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

Fix the stuck at startup causing 502 bad gateway. #4021

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

Conversation

jqknono
Copy link

@jqknono jqknono commented Sep 26, 2024

Some cloud vendors may have been blocked by AWS, this led to stuck at startup.

@jqknono
Copy link
Author

jqknono commented Sep 26, 2024

There are some issues about 502 bad gateway, cannot login.
The direct cause is the NPM getting stuck at startup, some of them related to the fetching of https://ip-ranges.amazonaws.com/ip-ranges.json

Some cloud vendors like Tencentcloud may block aws or be block by aws. Also , it may take a long time downloading this file, which blocked the port 81 backend.

#3965
#3585
#3574
#3157

@jqknono jqknono changed the title Fix the stuck at startup Fix the stuck at startup causing 502 bad gateway. Sep 26, 2024
@nginxproxymanagerci
Copy link

Docker Image for build 2 is available on DockerHub as jc21/nginx-proxy-manager:github-pr-4021

Note: ensure you backup your NPM instance before testing this PR image! Especially if this PR contains database changes.

@@ -15,7 +15,6 @@ async function appStart () {
.then(() => {
return apiValidator.loadSchemas;
})
.then(internalIpRanges.fetch)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removing this is not good enough to fix the problem. This call might be in the wrong place but you haven't put a better one in the pr.

@yangluoning
Copy link

@jc21 develop branch dont have your commit,Is it flushed by other commit?

@jc21
Copy link
Member

jc21 commented Dec 16, 2024

@jc21 develop branch dont have your commit,Is it flushed by other commit?

I'm not sure what "commit" you're talking about? This PR is not mine and is not merged either.

@jc21 jc21 added the bug label Dec 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants