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

Add Environment Variable to allow for Raytha to run at a different path base #44

Closed
apexdodge opened this issue Jan 17, 2023 · 4 comments
Labels
enhancement New feature or request
Milestone

Comments

@apexdodge
Copy link
Contributor

If you want to run Raytha behind a reverse proxy where your domain might be something.com but you want Raytha to run at something.com/mysite, it works remarkably well. I tested it on kubernetes behind nginx ingress.

All you have to do is add this line of code in Startup.cs as the first line in Configure(..)

image

But I think it might a good idea to allow this to be specified in an appsettings.json.

Task:

  • Add in appsettings.json a new app setting: "PATHBASE". If the value is set in appsettings.json, change the PathBase to the one specified, otherwise leave it be.
@apexdodge apexdodge added enhancement New feature or request good first issue Good for newcomers labels Jan 17, 2023
@apexdodge apexdodge added this to the v0.9.4 milestone Jan 17, 2023
@apexdodge
Copy link
Contributor Author

Live website link needs to take into account basepath
image

@apexdodge
Copy link
Contributor Author

Probably some areas that were not properly using RedirectToAction like on login:

image

@apexdodge apexdodge removed the good first issue Good for newcomers label Jan 17, 2023
@apexdodge apexdodge modified the milestones: v0.9.4, v1.0.0 Jan 17, 2023
@apexdodge apexdodge removed this from the v1.0.0 milestone Feb 5, 2023
@apexdodge apexdodge added this to the v1.0.1 milestone Feb 13, 2023
apexdodge added a commit that referenced this issue Feb 18, 2023
@apexdodge
Copy link
Contributor Author

Implemented, needs testing

apexdodge added a commit that referenced this issue Feb 19, 2023
apexdodge added a commit that referenced this issue Feb 19, 2023
@apexdodge
Copy link
Contributor Author

PathBase was missed for these two js controllers:

image

@apexdodge apexdodge reopened this Mar 4, 2023
apexdodge added a commit that referenced this issue Mar 4, 2023
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

1 participant