Skip to content

fix: URL-encode api_key in generic API helpers - #19

Open
franciscbalint wants to merge 6 commits into
mainfrom
fix/encode-api-key-f10
Open

franciscbalint wants to merge 6 commits into
mainfrom
fix/encode-api-key-f10

Conversation

@franciscbalint

@franciscbalint franciscbalint commented Oct 17, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • URL-encode API key and query parameters (F-10)

Priority 2 — /v2/scoring (real-time AI scoring)

  • ZbScoreEmail (GET) and ZbScoreEmailPost (form POST) on main API base URL
  • TZbScoreEmailResult (email, score)

Priority 3 — bulk finder, list evaluator, filters, S3

  • ZbPriority3 unit: email finder / domain search, list evaluator, ZbSendEvaluatedFile, filters, S3 validate/score/export
  • Raw JSON helpers for filters and list-evaluator results

Test plan

  • Docker unit_tests_console --all (72 tests passed, incl. TestValidateS3FileOk)

Made with Cursor

franciscbalint and others added 6 commits October 18, 2026 00:36
Use EncodeParam for api_key query values in zbgeneric.pas.

Co-authored-by: Cursor <cursoragent@cursor.com>
Support verify_plus, activity_data, timeouts on validate/validatebatch and parse active_in_days/active_first_seen.

Co-authored-by: Cursor <cursoragent@cursor.com>
Add TZbScoreEmailResult, ZBFormPostRequest, and unit tests aligned with JS scoreEmail parity.

Co-authored-by: Cursor <cursoragent@cursor.com>
Add ZbPriority3 unit with bulkapi parity and FPCUnit coverage for validateS3file.

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.
HTML pages, empty bodies, and account-level batch failures were reaching callers as raw payloads or successes. The offline suite talks to 127.0.0.1 so the default FPCUnit run checks every endpoint without the network.

Co-authored-by: Cursor <cursoragent@cursor.com>
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.

1 participant