-
Notifications
You must be signed in to change notification settings - Fork 53
docs(titkok): clarify media posting requirements #177
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
Conversation
@ethrx is attempting to deploy a commit to the Listinai Team on Vercel. A member of the Team first needs to authorize it. |
WalkthroughAdded a warning Callout to the TikTok provider docs noting that TikTok fetches media via pull_from_url, requires publicly reachable HTTPS media (localhost/private routes will fail), and advising use of a reverse proxy (e.g., Caddy) or public object storage/CDN and verifying the media domain in the TikTok developer account. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
Tip 👮 Agentic pre-merge checks are now available in preview!Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.
Example: reviews:
pre_merge_checks:
custom_checks:
- name: "Undocumented Breaking Changes"
mode: "warning"
instructions: |
Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal). Please share your feedback with us on this Discord post. 📜 Recent review detailsConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
Pre-merge checks (2 passed, 1 warning)❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✨ Finishing Touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (2)
pages/providers/tiktok.mdx (2)
18-22
: Clarify hosting scope (media vs app) and fix grammar.The callout currently conflates hosting Postiz itself with hosting the media files TikTok pulls. Also “an more manual” → “a more manual.” Suggested rewrite tightens the guidance and keeps it media-specific.
Apply this diff:
-<Callout type="warning"> -**NOTE:** Due to TikTok's restrictions, to upload media you have to host Postiz on a domain using a reverse proxy such as [Caddy](/reverse-proxies/caddy/) or an more manual setup with [Cloudflare R2](/configuration/r2/). - -Make sure the domain you host media on is on your developer account's list of sites with verified ownership. -</Callout> +<Callout type="warning"> +**NOTE:** TikTok fetches media via pull_from_url. Your media files must be publicly reachable over HTTPS; localhost or private routes (e.g., `/uploads`) will fail. Expose your uploads via a reverse proxy (e.g., [Caddy](/reverse-proxies/caddy/)) or use object storage/CDN such as [Cloudflare R2](/configuration/r2/) with public access. + +Ensure the media domain is listed under your TikTok developer account’s verified sites. +</Callout>
12-16
: Clarify TikTok redirect-URI and media-host verification requirements
- Line 15: revise the HTTPS callout to specify that Web Login Kit redirect URIs must begin with https only, and that only the Desktop Login Kit permits
http://localhost:‹port›
for development.- Lines 18–22: add a note that when using the Content Posting API’s
pull_from_url
, the media host domain (or URL prefix) must be pre-verified in your TikTok developer app.- Lines 47–49: update or remove the
http://localhost
example to indicate it applies solely to Desktop Login Kit development, not Web Login Kit.
Currently, Postiz only uses pull_from_url for TikTok content uploading: here.
If you are hosting your uploaded media on your localhost, or just haven't publicly exposed your
/uploads
route, TikTok will not be able to access your file.There is a viable alternative, work with chunks which would take a little bit of work but would bypass the need to publicly expose your
/uploads
.Anyways, this pull request clarifies the docs with the message:
Summary by CodeRabbit