Warning: Undefined array key "end_session_endpoint" since 3322786

This is actually the last item preventing us from completing a migration to D10, so I think the priority needs to be increased.

@srees, can you please elaborate a bit on your feedback?

Have you tested patch #3?

Have you tried setting the value of this field and exporting the config with the end_session_endpoint value?

First, I'll apologize for failing to see patch 3.

That said, I applied it, and the error changes now to:
Warning
: Undefined array key "upn" in
Drupal\openid_connect_windows_aad\Plugin\OpenIDConnectClient\WindowsAad->buildUserinfo()
(line
475
of
modules/contrib/openid_connect_windows_aad/src/Plugin/OpenIDConnectClient/WindowsAad.php
)

Not sure if that belongs in a new ticket since it's a different error now.

@srees, maybe add the fix for line 475, take inspiration from patch #3, generate a new patch.

Here's how to create a patch: → https://www.drupal.org/patch/create →

With that said, you may be able to adjust your configuration through the ui.

I would suggest visiting this page:

/admin/config/people/openid-connect/windows_aad/edit

and this page

/admin/config/people/openid-connect/settings

On both of these pages, click the save button.

This should fix the glitch.

You may want to do that a few times.

What version of openid_connect_windows_aad are you using?

What version of drupal core are you using? D10.0.11 ? D10.1.6? D9.5.11 ?

What version of openid_connect are you using?

I think we should update the configuration again and fix there bug there. I think its not required to add ?? '' in a few places to fix this bug.

chetan 11 → made their first commit to this issue’s fork.

Hi,
I have fixed the above error on the "OpenID Connect Windows Azure Active Directory" module, please check the raised MR.
Thanks.

I reviewed the change and tested changing the settings in our environment where we got the error.

We did get a lot of new settings both for the openid_connect.settings.yml and openid_connect.client.windows_aad.yml.

The code change looks good for me to take into account the missing configuration but I was also wondering if we should add the need configurations to the schema file:
https://git.drupalcode.org/project/openid_connect_windows_aad/-/blob/2.0...

To me it seems like the following configurations are missing from the schema:

redirect_url
enable_single_sign_out
iss_allowed_domains
end_session_endpoint
front_channel_logout_url

webflo → changed the visibility of the branch 3400940-fix-warning to hidden.

webflo → changed the visibility of the branch 3400940-fix-warning to active.

@heikkiy I updated the config schema. Most of the properties you had in your list does not exist anymore. enable_single_sign_out got converted in openid_connect_windows_aad_update_9205 but we did not remove it from the config, but that is another issue.

Automatically closed - issue fixed for 2 weeks with no activity.