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

#352 - Add deprecation wraning to the 1.x codebase #353

Merged
merged 1 commit into from
Oct 17, 2024

Conversation

Kamforka
Copy link
Collaborator

As per #352

@Kamforka Kamforka added the 1.x label Oct 17, 2024
@Kamforka Kamforka added this to the 1.8.2 milestone Oct 17, 2024
@Kamforka Kamforka self-assigned this Oct 17, 2024
@Kamforka
Copy link
Collaborator Author

Kamforka commented Oct 17, 2024

It will produce warnings like the belows:

$ python -c "import thehive4py"
<string>:1: DeprecationWarning: thehive4py v1.x is not maintained anymore as TheHive v3 and v4 are end of life. If you are using TheHive v5 please consider switching to thehive4py v2.x (https://github.com/TheHive-Project/TheHive4py)!

or

$ python -c "from thehive4py.models import Case"
<string>:1: DeprecationWarning: thehive4py v1.x is not maintained anymore as TheHive v3 and v4 are end of life. If you are using TheHive v5 please consider switching to thehive4py v2.x (https://github.com/TheHive-Project/TheHive4py)!

@nadouani
Copy link
Contributor

@Kamforka I believe this might break scripts that are supposed to just print a json to the console. Thoughts?

@Kamforka
Copy link
Collaborator Author

@Kamforka I believe this might break scripts that are supposed to just print a json to the console. Thoughts?

@nadouani yes, free text parsers would be affected by this change. But I'd consider those as do-it-for-your-own-risk solutions.

Nevertheless, do you think we should drop this PR then?

@nadouani
Copy link
Contributor

I would say GO, users can always downgrade to 1.8.1

@nadouani nadouani merged commit c148f5d into ZzZ-1.x Oct 17, 2024
@Kamforka Kamforka deleted the 352-add-depracation-warning-to-v1-codebase branch October 18, 2024 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants