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

FIX: Fixes Blueliv collector requirements #2161

Merged

Conversation

gethvi
Copy link
Contributor

@gethvi gethvi commented Mar 23, 2022

This PR fixes the following error (when running pip3 install -r intelmq/bots/collectors/blueliv/REQUIREMENTS.txt):

Collecting git+git://github.com/Blueliv/api-python-sdk (from -r /tmp/intelmq/intelmq/bots/collectors/blueliv/REQUIREMENTS.txt (line 1))
  Cloning git://github.com/Blueliv/api-python-sdk to /tmp/pip-req-build-uiy3i5_1
  Running command git clone -q git://github.com/Blueliv/api-python-sdk /tmp/pip-req-build-uiy3i5_1
  fatal: remote error:
    The unauthenticated git protocol on port 9418 is no longer supported.
  Please see https://github.blog/2021-09-01-improving-git-protocol-security-github/ for more information.
ERROR: Command errored out with exit status 128: git clone -q git://github.com/Blueliv/api-python-sdk /tmp/pip-req-build-uiy3i5_1 Check the logs for full command output.

@gethvi gethvi changed the title FIX: Fixes Blueliv collector requirements. FIX: Fixes Blueliv collector requirements Mar 23, 2022
@codecov-commenter
Copy link

codecov-commenter commented Mar 23, 2022

Codecov Report

Merging #2161 (9e90d0a) into develop (ce60fd4) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #2161   +/-   ##
========================================
  Coverage    76.34%   76.34%           
========================================
  Files          441      441           
  Lines        23649    23649           
  Branches      3739     3739           
========================================
  Hits         18055    18055           
  Misses        4857     4857           
  Partials       737      737           

Copy link
Contributor

@waldbauer-certat waldbauer-certat left a comment

Choose a reason for hiding this comment

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

LGTM

@waldbauer-certat waldbauer-certat merged commit 6e1c1bf into certtools:develop May 4, 2022
@sebix
Copy link
Member

sebix commented May 4, 2022

Thanks.

The changelog entry is missing

@sebix sebix added this to the 3.1.0 milestone May 4, 2022
@sebix sebix added bug Indicates an unexpected problem or unintended behavior component: bots labels May 4, 2022
gutsohnCERT added a commit to gutsohnCERT/intelmq that referenced this pull request May 4, 2022
gutsohnCERT added a commit to gutsohnCERT/intelmq that referenced this pull request May 4, 2022
sebix pushed a commit that referenced this pull request May 6, 2022
@gethvi gethvi deleted the fix-blueliv-requirements branch November 10, 2022 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Indicates an unexpected problem or unintended behavior component: bots
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants