Skip to content
This repository has been archived by the owner on Jul 31, 2024. It is now read-only.

How to customize the route path? #593

Closed
atpyk opened this issue Dec 16, 2016 · 13 comments
Closed

How to customize the route path? #593

atpyk opened this issue Dec 16, 2016 · 13 comments
Assignees

Comments

@atpyk
Copy link

atpyk commented Dec 16, 2016

I don't want to expose the default route path like "connect/token". Does it have any way to change it when inject the IdentityServer.
If no, do you guys plan to support this extension feature in future?

@leastprivilege
Copy link
Member

What do you want to expose instead and why?

@atpyk
Copy link
Author

atpyk commented Dec 16, 2016

I prefer to use other route path for my project, I hope it would be customized, like "/oauth2/token".

@leastprivilege
Copy link
Member

no. not right now. and no plans either.

It is not that a human needs to type them - only machines.

@atpyk
Copy link
Author

atpyk commented Dec 16, 2016

Okay, thanks for your feedback.

@atpyk
Copy link
Author

atpyk commented Dec 16, 2016

To be honest, our case is, we have lots of sub systems need to do authentication via accessing an original authentication service that expose the route path is "/oauth2/token".
For now, we want to use identityserver4 instead, meanwhile we don't want to change the original contracts, we'd like to customize the route path of IdentityServer to adapt our contracts.
That's the context.

@leastprivilege
Copy link
Member

Understood. But we don't allow that right now. It is a bit more complex than it sounds.

The right way would be to get the real token endpoint from the discovery endpoint.

@atpyk
Copy link
Author

atpyk commented Dec 16, 2016

I've went through the IdentityServer code. It seems like all the Endpoints those are depend on the enum "EndpointName", not const strings, and actually, you transfer an dictionary(Constants.EndpointPathToNameMap) when construct EndpointRouter, perhaps you could expose a property that could be set outside, and replace this dictionary in constructor. Does it make sense?

@leastprivilege
Copy link
Member

We already went through that exercise and there are a couple of complications - CORS, endpoint with mutliple sub-endpoints (mostly discovery and UI related).

I added a feature request/backlog label to it. It def. won't happen for RTM (next week).

@leastprivilege
Copy link
Member

Feel free to do a POC and let us know.

@brockallen
Copy link
Member

I'm going to close, as it seems to be a dup of either #19 or #442. Feel free to join in on those threads.

@danielmeza
Copy link

What about a simply redirect on IIS?

@312527814
Copy link

I hope the identityServer4 can support customize the route path extension feature in future

  1. in old system we can't change the original contracts path
  2. it good for data distribution in Reverse proxy
  3. it friendly to a clearer description of the business
  4. so on
    expect it can support customize the route path extension feature in future!

@lock
Copy link

lock bot commented Jan 11, 2020

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Jan 11, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants