Skip to content

fix: redact API key in RestClient errors and harden publish CI - #27

Open
franciscbalint wants to merge 7 commits into
developfrom
fix/redact-api-key-publish-hardening
Open

franciscbalint wants to merge 7 commits into
developfrom
fix/redact-api-key-publish-hardening

Conversation

@franciscbalint

@franciscbalint franciscbalint commented Oct 17, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Redact configured API key from RestClient error messages in BaseRequest.
  • Set bundler-cache: false on publish workflow test/publish Ruby setup steps.
  • Priority 3 bulk APIs including list_s3_files.

Polish (latest)

  • WebMock spec for list_s3_files (/v2/listS3files).
  • bulk_get returns JSON arrays (needed for listS3files).

Test plan

  • bundle exec rspec spec/priority3_spec.rb (3 examples)

Made with Cursor

franciscbalint and others added 7 commits October 18, 2026 00:36
Wrap RestClient failures with redacted messages and disable bundler-cache on
publish workflow jobs to match npm publish hardening.

Co-authored-by: Cursor <cursoragent@cursor.com>
Optional keyword params on validate and validate_batch per ZeroBounce v2 docs.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Add bulk email finder, domain search, list evaluator, filters, and S3 helpers with WebMock-backed RSpec coverage.

Co-authored-by: Cursor <cursoragent@cursor.com>
Add form-encoded POST option on single validate (alongside GET) and GET
listS3files on bulkapi root, with unit tests and API parity doc updates.
listS3files returns a JSON array; allow bulk_get to return non-hash payloads so the new priority3 spec can run.

Co-authored-by: Cursor <cursoragent@cursor.com>
HTML and Cloudflare bodies were raised in full, an account-level batch failure looked successful, and an explicit false header flag was left off the request.

Co-authored-by: Cursor <cursoragent@cursor.com>
end

def html_title(body)
match = body.to_s.match(%r{<title[^>]*>(.*?)</title>}im)

This branch has not been deployed

No deployments
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