Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

IDEV-2272 and IDEV-2273: support header auth for iris endpoints and change default Feeds auth behavior#178

Merged
jbabac merged 5 commits intorelease-2.5.3DomainTools/python_api:release-2.5.3from
IDEV-2273-support-header-auth-for-iris-endpointsDomainTools/python_api:IDEV-2273-support-header-auth-for-iris-endpointsCopy head branch name to clipboard
Oct 8, 2025
Merged

IDEV-2272 and IDEV-2273: support header auth for iris endpoints and change default Feeds auth behavior#178
jbabac merged 5 commits intorelease-2.5.3DomainTools/python_api:release-2.5.3from
IDEV-2273-support-header-auth-for-iris-endpointsDomainTools/python_api:IDEV-2273-support-header-auth-for-iris-endpointsCopy head branch name to clipboard

Conversation

@jbabac
Copy link
Contributor

@jbabac jbabac commented Oct 7, 2025

Changes:

  1. Change default Feeds auth behavior
    a. always_sign_api_key is False by default unless explicitly set by user
    b. header_authentication is True by default unless explicitly set by user
    c. Since RTTF endpoints don’t support signed API keys with any auth method, if the user sets always_sign_api_key=True for RTTF, endpoints throw an error that says “Real Time Threat Feeds do not support signed API keys.”
    d. For non-RTTF endpoints, always_sign_api_key is set True unless explicitly set by user

  2. Support X-Api-Key Header Auth for Iris Endpoints

Unit Tests:
image

E2E Tests:
image

@jbabac jbabac self-assigned this Oct 7, 2025
Copy link
Contributor

@wesleya wesleya left a comment

Choose a reason for hiding this comment

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

Nice job JD!

@jbabac jbabac merged commit 78c93ed into release-2.5.3 Oct 8, 2025
12 checks passed
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.