-
Notifications
You must be signed in to change notification settings - Fork 15
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
remove detectionlists, high-risk-employee, and departing-employee cli… #458
Conversation
b0487fd
to
0b931c1
Compare
@@ -8,6 +8,13 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 | |||
The intended audience of this file is for py42 consumers -- as such, changes that don't affect | |||
how a consumer would use the library (e.g. adding unit tests, updating documentation, etc) are not captured here. | |||
|
|||
## 1.27.0 - 2023-08-04 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We removed APIs in this update, I understand they were already deprecated, but should this be a major version bump?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My thought was no because the backend doesn't exist anyway, so updating versions or pinning an old version would have no effect.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Honestly its closer to a bugfix chore imo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But idk
There's one more reference to detection lists in docs/userguides/basics.md. |
src/py42/sdk/__init__.py
Outdated
cases_conn = Connection.from_microservice_key(kv_service, cases_key, auth=main_auth) | ||
trusted_activities_conn = Connection.from_microservice_key( | ||
kv_service, trusted_activities_key, auth=main_auth | ||
) | ||
watchlists_conn = Connection.from_microservice_key( | ||
kv_service, watchlists_key, auth=main_auth | ||
) | ||
userriskprofile_svc = UserRiskProfileService(watchlists_conn) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider user_risk_profile_svc
.
src/py42/sdk/__init__.py
Outdated
cases_conn = Connection.from_microservice_key(kv_service, cases_key, auth=main_auth) | ||
trusted_activities_conn = Connection.from_microservice_key( | ||
kv_service, trusted_activities_key, auth=main_auth | ||
) | ||
watchlists_conn = Connection.from_microservice_key( | ||
kv_service, watchlists_key, auth=main_auth | ||
) | ||
userriskprofile_svc = UserRiskProfileService(watchlists_conn) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider user_risk_profile_svc
.
tests/services/test_alert_rules.py
Outdated
mocker, "{}" | ||
) | ||
return detection_list_user_service | ||
def mock_user_service(mocker): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider mock_user_risk_profile_service
d79d741
to
d5bf66f
Compare
d5bf66f
to
07138fe
Compare
No description provided.