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.

Sign up for GitHub

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

Jump to bottom

CLI - Implement Dynamic Ports for Ory Redirect #21

Open
Abraxas-365 opened this issue Jul 31, 2024 · 5 comments
Open

CLI - Implement Dynamic Ports for Ory Redirect #21

Abraxas-365 opened this issue Jul 31, 2024 · 5 comments
Assignees
@Abraxas-365
Labels
enhancement New feature or request

Comments

@Abraxas-365
Copy link
Contributor

No description provided.

@Abraxas-365 Abraxas-365 self-assigned this Jul 31, 2024
@Abraxas-365 Abraxas-365 added the enhancement New feature or request label Jul 31, 2024
@Abraxas-365 Abraxas-365 changed the title CLI - Implement Dynamic Ports CLI - Implement Dynamic Ports for Ory redirectt Jul 31, 2024
@Abraxas-365 Abraxas-365 changed the title CLI - Implement Dynamic Ports for Ory redirectt CLI - Implement Dynamic Ports for Ory redirect Jul 31, 2024
@Abraxas-365 Abraxas-365 changed the title CLI - Implement Dynamic Ports for Ory redirect CLI - Implement Dynamic Ports for Ory Redirect Jul 31, 2024
@osbornm
Copy link
Member

osbornm commented Aug 21, 2024

This is a security concern, we do not want to support dynamic ports (in most cases) can you please tell me more about why you need this or I'll go ahead and close this issue.

@Abraxas-365
Copy link
Contributor Author

If the port we are using is already in use, the user will not be able to log in. Postman and other applications have dynamic ports to log in.

@Abraxas-365
Copy link
Contributor Author

We can add a disclaimer though

@osbornm
Copy link
Member

osbornm commented Aug 21, 2024

okay so let's pick 2-3 options while add those and we can choose form that list. I would like it to be a known list of ports.

@Abraxas-365
Copy link
Contributor Author

Currently, we are using redirectURI = "http://localhost:5001/token". What do you think about: 5001, 5025, 5030

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Abraxas-365 Abraxas-365

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@osbornm @Abraxas-365