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

Update Program.cs #93

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

mistermik
Copy link

App always redirect to localhost and not the current domain so, this does not work when using codespaces as the callback is a public domain. Fixing this problem by using UseForwardedHeaders, you ensure that your application correctly handles requests in scenarios where it is behind a proxy, which can affect URL generation and redirection.

App always redirect to localhost and not the current domain so, this does not work when using codespaces as the callback is a public domain.
Fixing this problem  by using UseForwardedHeaders, you ensure that your application correctly handles requests in scenarios where it is behind a proxy, which can affect URL generation and redirection.
@mistermik mistermik requested a review from a team as a code owner January 26, 2024 04:33
@frederikprijck
Copy link
Contributor

frederikprijck commented Jan 26, 2024

Thanks, as mentioned, this sample is to help quickstart people with setting up auth0 in asp.net. Running things behind a proxy and whatnot is confusing for those not familiar with the concept, is not relevant to setting up auth0, and people will just copy this in their production app.

We want to keep it as minimal, yet useful as possible.

Closing

@mistermik
Copy link
Author

@frederikprijck CIC certification demands customers to pick up a quickstart and use github codespaces; however, if customers choose the .NET sample, this will fail because the request will always redirect back to localhost as CallBack. If you search online, you will find a number of issues with this sample raised in the Okta developer community. That is why I am proposing the change. At the very least, you can check to see if codespaces are being used and, if so, make the changes I suggested. My two cents on making a sample work for anyone taking CIC training or certification without wasting time on fixing something that should just work out of the box.

@frederikprijck
Copy link
Contributor

Thanks for that context!

I was not aware of the sample being used in that context, will need to look into that to understand more.

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

Successfully merging this pull request may close these issues.

None yet

2 participants