Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[anyrun-feed] Fixed paid auth #3382

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

rnaumov-any
Copy link
Contributor

Any.Run provides a demo feed with API-key authorization and a Paid feed with Basic authorization. Added automatic selection of authorization method

Proposed changes

  • Enumeration of authorization methods

Related issues

Checklist

  • I consider the submitted work as finished
  • I tested the code for its functionality using different use cases
  • I added/update the relevant documentation (either on github or on notion)
  • Where necessary I refactored code to improve the overall quality

Further comments

@richard-julien richard-julien force-pushed the master branch 2 times, most recently from 9818df2 to 0f4b30d Compare February 4, 2025 14:35
@flavienSindou flavienSindou added community use to identify PR from community filigran support [optional] use to identify an issue related to feature developed & maintained by Filigran. do not merge Do not merge this PR until this tag will be removed labels Feb 4, 2025
@helene-nguyen helene-nguyen removed the filigran support [optional] use to identify an issue related to feature developed & maintained by Filigran. label Feb 5, 2025
@helene-nguyen helene-nguyen changed the title "[connector-anyrun-feed] Fixed paid auth. [anyrun-feed] Fixed paid auth Feb 6, 2025
@SamuelHassine SamuelHassine force-pushed the master branch 2 times, most recently from b513b96 to 11a05bf Compare February 14, 2025 12:49
@rnaumov-any
Copy link
Contributor Author

hello, why our fix is not merging? this fix is really important, our client can't use our feed. Old version after this change is not working

@rnaumov-any
Copy link
Contributor Author

We've updated the documentation. In accordance with the documentation, Basic authorization must be used https://intelligence.any.run/Threat_Intelligence_Feeds_Documentation_v3.pdf

@helene-nguyen helene-nguyen self-assigned this Feb 28, 2025
@helene-nguyen
Copy link
Member

@rnaumov-any Thanks for the fix, merge is blocked because format and commits are not signed. Could you please ensure the format, you can rebase master as black and isort was updated to a new version and could you please sign your commits with GPG to have a verified signature ?

@helene-nguyen helene-nguyen removed the do not merge Do not merge this PR until this tag will be removed label Feb 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community use to identify PR from community
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants