Closed (fixed)
Project:
OpenID Connect Microsoft Azure Active Directory client
Version:
2.0.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
20 Nov 2022 at 21:25 UTC
Updated:
10 Nov 2023 at 23:29 UTC
Jump to comment: Most recent
Comments
Comment #3
webflo commentedComment #5
moshe weitzman commentedI am seeing a warning when this setting is not populated. This is optional, right?
Comment #6
moshe weitzman commentedMy fault. I was missing a config key.
Comment #7
rosk0I'm seeing the same warning now and I think that shouldn't be happening .
Update
openid_connect_windows_aad_update_9205()introduced here was only updating configurations where->condition('settings.enable_single_sign_out', TRUE, '='). To me this means there is a bug, either in the update code or in the main module which tries to use optional property without checking for existence first.Can we please have a clarification if this new "end_session_endpoint" property is optional or now.
Also, it wasn't added to the config schema here. I believe this should be re-opened and worked on , but don't have permission to do so.
Comment #8
dreg commentedBumping as I'm seeing the same issue. If set to empty I get the same:
Warning: Undefined array key "end_session_endpoint" in Drupal\openid_connect_windows_aad\Plugin\OpenIDConnectClient\WindowsAad->getEndpoints() (line 278 of modules/contrib/openid_connect_windows_aad/src/Plugin/OpenIDConnectClient/WindowsAad.php).
Comment #9
heikkiy commentedEncountering this also.
We updated our site to Drupal 10 from Drupal 9 and at the same time updated OpenID Connect to 3.0 alpha2 and OpenID Connect Microsoft Azure AD client to 2.0.0-beta5. Database updates and configurations are updated but after logging in I get the same warning in the page. Tried running drush updb and drush cim again but there are no changes incoming.
It seems like this issue should be reopened.
Here is the complete stacktrace
Comment #10
joseph.olstadYes I also noticed this today and we're using the latest release of openid_connect_windows_aad 2.0.0-beta6 and we're also using openid_connect 3.0.0-alpha2
Comment #11
joseph.olstad#3400940: Warning: Undefined array key "end_session_endpoint" since 3322786
Comment #12
joseph.olstadPlease review possible solutions in:
#3400940: Warning: Undefined array key "end_session_endpoint" since 3322786