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 missing f in f-string #430

Merged
merged 1 commit into from Aug 1, 2019
Merged

Fix missing f in f-string #430

merged 1 commit into from Aug 1, 2019

Conversation

paalbra
Copy link
Contributor

@paalbra paalbra commented Aug 1, 2019

Fixes: #429

Same request as in the issue now produces:

GET /servers/getPyMISPVersion.json HTTP/1.1
Host: localhost:1337
User-Agent: PyMISP 2.4.111.2 - Python 3.7
Accept-Encoding: gzip, deflate
Accept: application/json
Connection: keep-alive
Authorization: asdf
content-type: application/json

@Rafiot
Copy link
Member

Rafiot commented Aug 1, 2019

.... That's a stupid bug, thank you very much for finding and fixing it.

@Rafiot Rafiot merged commit a3a6e3c into MISP:master Aug 1, 2019
@codecov
Copy link

codecov bot commented Aug 1, 2019

Codecov Report

Merging #430 into master will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #430   +/-   ##
=======================================
  Coverage   39.94%   39.94%           
=======================================
  Files          34       34           
  Lines        5881     5881           
=======================================
  Hits         2349     2349           
  Misses       3532     3532
Impacted Files Coverage Δ
pymisp/aping.py 12.92% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b1c79f9...8d302d3. Read the comment docs.

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.

User-Agent is python code
2 participants