G
YouTube TikTok Games
Skip to content

README fixes, license copyright, SECURITY.md - #46

Merged
tomasr78 merged 1 commit into
mainfrom
readme-audit-license-security
Nov 12, 2026
Merged

tomasr78 merged 1 commit into
mainfrom
readme-audit-license-security

Conversation

@tomasr78

Copy link
Copy Markdown
Contributor

README bug audit, like the one done for the .NET SDK, plus a security policy.

Tested in ruby:3.4-slim against production: examples/create_pdf_thumbnail.rb (a 231x300 JPG), with the test files deleted afterwards. The Pdfa parameter was tested with the Python and PHP SDKs (same API call).

No SDK code changes.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings November 12, 2026 09:29

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🔵 Needs a closer look

Maintainer confirmation is needed that replacing the original copyright attribution is authorized and private vulnerability reporting is enabled before making it the sole reporting route, as neither is established by the source changes.

No open issues 🎉

What changed in this PR

This documentation-only PR corrects README examples and links, adds a private vulnerability-reporting policy, and updates the MIT license copyright notice. No SDK code changes are included.

Changes:

  • Fix Ruby syntax in conversion examples and replace PdfResolution with Pdfa.
  • Refresh documentation links and add security reporting and supported-version guidance.
  • Update the license copyright to 2012–2026 ConvertAPI.
File Description
SECURITY.md Adds private vulnerability-reporting instructions and security support guidance.
README.md Corrects conversion examples and updates documentation and Contributor Covenant links.
LICENSE.txt Updates the copyright notice, heading, and line wrapping.

Review effort: Balanced

@tomasr78
tomasr78 merged commit 1f6407b into main Nov 12, 2026
6 checks passed
@tomasr78
tomasr78 deleted the readme-audit-license-security branch November 12, 2026 09:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants