search

simevo / spid-wordpress

SPID (Sistema Pubblico di Identità Digitale) WordPress plugin
GNU Affero General Public License v3.0
4 stars 1 forks source link

configure correctly the entityId of the test IdP #52

Open simevo opened 6 years ago

simevo commented 6 years ago

A default value for the entityId of the test IdP is currenty hardwired here: https://github.com/simevo/spid-wordpress/blob/master/templates/spid-button.php#L44

While this setting is OK for the usual docker-compose setup we use in development, it will not work for most people.

The sp_idp option has been envisaged to allow entering the entityId of the actual test IdP instace used. The code above should use that option.

michaeltieso commented 6 years ago

Do you mean that this should be an option that can be changed in the admin settings?

simevo commented 6 years ago

Yes, we already have it ! screenshot_20181029_124858

michaeltieso commented 6 years ago

I'll get this working today. In the meantime, I'll need to merge the new admin section first.

simevo commented 6 years ago

This is mostly done, but we are blocked by #73 ; deferring to milestone 0.2