Skip to content

README fixes, MIT license GitHub can detect, SECURITY.md - #129

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 license file and a security policy.

The Pdfa parameter was tested against production with the Python SDK (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.

🟡 Changes recommended

The LICENSE change removes existing Baltsoft and Tomas Rutkauskas copyright notices that the MIT license requires to be preserved alongside the new ConvertAPI notice.

1 open issue
What changed in this PR

This PR updates the Node.js SDK’s documentation and repository policies without changing SDK code.

Changes:

  • Corrects README conversion parameters, package usage, and documentation links.
  • Standardizes the MIT license layout for GitHub detection.
  • Adds private vulnerability reporting instructions and a security-fix support policy.
File Description
SECURITY.md Adds vulnerability reporting guidance and supported-version policy.
README.md Fixes examples and refreshes documentation links.
LICENSE Standardizes MIT formatting and updates the copyright notice.

Review effort: Balanced

Comment thread LICENSE
ConvertAPI.Node
https://github.com/ConvertAPI/convertapi-node
Copyright (c) 2018 Tomas Rutkauskas
Copyright (c) 2012-2026 ConvertAPI
@tomasr78
tomasr78 merged commit 88e6a83 into main Nov 12, 2026
5 checks passed
@tomasr78
tomasr78 deleted the readme-audit-license-security branch November 12, 2026 09:43
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