Repository navigation
fix: correct Docker redirect URI port in provider docs (5000 -> 4007) - #258
Closed
thisismairaj wants to merge 1 commit into
Closed
thisismairaj wants to merge 1 commit into
thisismairaj wants to merge 1 commit into
Conversation
The official postiz-docker-compose setup maps the app to host port 4007 (docker-compose.yaml: "4007:5000"), matching MAIN_URL/FRONTEND_URL in that same file. Every provider doc's "Docker" redirect URI example instead showed localhost:5000, the container's internal port, which a browser following the standard Quick Start setup can't actually reach. Verified by self-hosting via postiz-docker-compose and hitting this exact mismatch while registering a LinkedIn OAuth app.
|
Hi @thisismairaj! Thanks for the PR. Contributions to Postiz are gated behind an application. Please apply at https://contribute.postiz.com/p/postiz and we'll reopen this PR once you're approved. Not sure what just happened? https://contribute.postiz.com/for-contributors This project also requires a signed Contributor License Agreement. You can sign it now at https://contribute.postiz.com/p/postiz/cla so you're ready as soon as your application is approved. |
|
Hi @thisismairaj! Your application for Postiz is awaiting review. We'll reopen this PR once it's approved. Status: https://contribute.postiz.com/p/postiz |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Every provider's self-host doc lists the Docker OAuth2 redirect URI as
http://localhost:5000/integrations/social/<provider>, but the official postiz-docker-compose setup (the one linked from the Quick Start) maps the app container to host port 4007, not 5000:Port 5000 is only the container's internal port. A browser following the standard Quick Start setup can't reach
localhost:5000directly, so every provider's documented Docker redirect URI is unreachable as written, and registering the redirect URI exactly as documented fails during the OAuth callback.Found this firsthand while self-hosting Postiz locally via
postiz-docker-composeand registering a LinkedIn developer app for OAuth.Changes
Updated the
Docker:redirect URI line in all 17 affected provider docs (18 occurrences, Instagram has two) fromlocalhost:5000tolocalhost:4007, matching the official compose file.tumblr.mdxalready had the correct port and was left untouched.Test plan
postiz-docker-compose'sdocker-compose.yaml("4007:5000",FRONTEND_URL: 'http://localhost:4007')localhost:5000URI, confirmedlocalhost:5000isn't reachable from the host/browsergrep-verified no remaininglocalhost:5000references acrossself-host/providers/