OpenConext / OpenConext-profile

Shows user's attributes and other profile information
Apache License 2.0
2 stars 0 forks source link

v1.1.0: 'My services'-page throws 'Server error' #103

Closed tvdijen closed 6 years ago

tvdijen commented 6 years ago

Mar 7 17:31:40 webapp-1 openconext-profile[30101]: request.CRITICAL: Uncaught PHP Exception OpenConext\Profile\Exception\AssertionFailedException: "Consent JSON structure must contain key "name_id_format"" at /apps/programs/openconext/profile/OpenConext-profile-1.1.0/vendor/beberlei/assert/lib/Assert/Assertion.php line 288 {"exception":"[object] (OpenConext\Profile\Exception\AssertionFailedException(code: 26): Consent JSON structure must contain key \"name_id_format\" at /apps/programs/openconext/profile/OpenConext-profile-1.1.0/vendor/beberlei/assert/lib/Assert/Assertion.php:288)"} []

thijskh commented 6 years ago

Did you upgrade Engineblock aswell?

tvdijen commented 6 years ago

I'm at 5.6.4 now and the release notes state 5.5.5 as a minimum

thijskh commented 6 years ago

It works for me with 5.6.4. Needs more investigation…

tvdijen commented 6 years ago

I'm pretty sure it relates to this commit https://github.com/OpenConext/OpenConext-profile/commit/144b1448cc600a375c10371406165b08f00115d4 Not sure why it affects me though... @jorissteyn Can you reason what's wrong here?

thijskh commented 6 years ago

NameID format is a mandatory field for an SP in both Manage and Serviceregistry. Do you perhaps somehow have an SP without a nameid format?

tvdijen commented 6 years ago

It was never mandatory in SR?? Anyway, I've set it on all SP's and it didn't solve the problem

jorissteyn commented 6 years ago

I'm at 5.6.4 now and the release notes state 5.5.5 as a minimum

That's the culprit. We maintain three versions of EngineBlock in parallel, so for version 1.1.0 of profile you need to update to one of the following versions of EngineBlock:

tvdijen commented 6 years ago

EB 5.6.5 was released a day after I filed this issue :') Oh well, I guess now that this is clear, this issue can be closed..