Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

metadata endpoint should be http://example.com/metadata #55

Open
simevo opened this issue Oct 27, 2018 · 3 comments
Open

metadata endpoint should be http://example.com/metadata #55

simevo opened this issue Oct 27, 2018 · 3 comments
Labels
enhancement New feature or request
Milestone

Comments

@simevo
Copy link
Owner

simevo commented Oct 27, 2018

... rather than http://example.com/wp-login.php?sso=spid&metadata

@simevo simevo added the enhancement New feature or request label Oct 27, 2018
@simevo simevo added this to the 0.2 milestone Oct 27, 2018
@michaeltieso
Copy link
Collaborator

I looked into this a bit but it can be a bit messy depending on a users permalink structure. Checks will need to be in place to make sure an existing permalink does not exist.

@giuliogatto
Copy link
Collaborator

giuliogatto commented Oct 29, 2018

That was one of my concerns in the issue at : #46
I think a WordPress fresh install doesn't have any active permalink setting , or am I wrong? This means that having a /metadata route/url cannot be assured all the time and cannot be enforced from inside the plugin

@simevo
Copy link
Owner Author

simevo commented Oct 29, 2018

so it could be a configurable permalink ... luckily this is for milestone 0.2, we should get past milestone 0.1 first !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants