Skip to content

Omit empty optional fields#36

Merged
Pajk merged 1 commit intov2from
browser-check-optional-fields
Apr 3, 2025
Merged

Omit empty optional fields#36
Pajk merged 1 commit intov2from
browser-check-optional-fields

Conversation

@Pajk
Copy link
Collaborator

@Pajk Pajk commented Apr 3, 2025

Resolves #ISSUE_NUMBER


Before the change?

  • Sending empty strings for optional fields domain (header, cookie) and path (cookie).

After the change?

  • Omit the field if the value is empty string.

Pull request checklist

  • Acceptance Tests have been updated, run (make testacc), and pasted in this PR (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)

Acceptance Test Output


Does this introduce a breaking change?

  • Yes
  • No

@Pajk Pajk merged commit 3a7186f into v2 Apr 3, 2025
4 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Apr 3, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants