-
Notifications
You must be signed in to change notification settings - Fork 327
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
URL rewriting issue after new Umbrella installation #192
Comments
As @jykae mentions, the /admin path is unavailable, rendering a 404. This is strange behavior, as it doesn't crop up until the first API backend is published. |
And to be precise, this is Nginx "not found" message, I get after API configuration publishing. |
Sorry for the trouble! Now that you mention this issue, I think I've possibly seen this before with the v0.8.0 package release. It hinges on a couple factors, and I'm pretty sure it got fixed later in git master, but that unfortunately doesn't help you with the package release. As a workaround can you try adding this to your hosts:
- hostname: YOUR_HOSTNAME_HERE
default: true Substitute I'll verify that this situation is indeed fixed in git master and add some test coverage around this scenario. We're also trying to wrap up a few things and the get the v0.9.0 package releases out in the next week or two, so hopefully that will also help more permanently solve this issue (but the above YAML should still be perfectly valid under v0.9.0). |
Thanks! Solved the issue. |
I installed Umbrella 0.8.0 from .deb package.
We are currently using SSL certs that came with the package.
When I publish API configuration, Umbrella rewrites URLs so that I am not able to access /admin path.
We have another server that has SSL certificates setup, it works.
Could this be better informed in API Umbrella documentation, bolded or something..
Currently it is in one sentence under "Installation":
"API Umbrella will use as self-signed certificate which won't be valid for production use."
I think it should be mentioned under "Getting started" as well.
The text was updated successfully, but these errors were encountered: