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

Extended dicom #24

Merged
merged 5 commits into from
Apr 8, 2024
Merged

Extended dicom #24

merged 5 commits into from
Apr 8, 2024

Conversation

jstucke
Copy link
Collaborator

@jstucke jstucke commented Mar 22, 2024

@jstucke jstucke requested a review from euwint March 22, 2024 15:11
@jstucke jstucke self-assigned this Mar 22, 2024
@euwint
Copy link
Collaborator

euwint commented Mar 27, 2024

  • honeypots --setup all --auto führt zu folgendem Fehler:

[ERROR] Starting honeypots failed: '>' not supported between instances of 'NoneType' and 'int'
Traceback (most recent call last):
File "/home/winter/Git/honeypots/venv/lib/python3.10/site-packages/honeypots/main.py", line 229, in _start_all_servers
self._start_server(service)
File "/home/winter/Git/honeypots/venv/lib/python3.10/site-packages/honeypots/main.py", line 261, in _start_server
status = server.run_server(process=True, auto=auto)
File "/home/winter/Git/honeypots/venv/lib/python3.10/site-packages/honeypots/base_server.py", line 108, in run_server
if port > 0:
TypeError: '>' not supported between instances of 'NoneType' and 'int'

Ohne --auto tritt der Fehler nicht auf.
Er tritt allerdings auch auf, wenn ich einen einzelnen Server (honeypots --setup dicom) starten möchte.
python dicom_server.py --auto --custom funktioniert wiederrum

@jstucke
Copy link
Collaborator Author

jstucke commented Apr 2, 2024

Ohne --auto tritt der Fehler nicht auf. Er tritt allerdings auch auf, wenn ich einen einzelnen Server (honeypots --setup dicom) starten möchte. python dicom_server.py --auto --custom funktioniert wiederrum

Looks like a return was missing -- whoopsie 🙈

@euwint euwint merged commit b51f57a into main Apr 8, 2024
5 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