Closed gavinelder closed 1 year ago
@chunyong-lin jira-cloud
SGTM I would like to see what #1311 and v4 look like before a major overhaul, however building and introducing before a major breaking change and removing old Outputs like PD v1, Jira, etc may make sense.
to: @ryandeivert @chunyong-lin @Ryxias cc : @blakemotl related to: https://github.com/airbnb/streamalert/pull/1311
Background
The following change was discussed on https://github.com/airbnb/streamalert/pull/1311 where the original Jira-v2 was just a heavy cut/paste job with some slight modifications, which is not ideal.
https://streamalert.slack.com/archives/C3BHE2Z0S/p1610399094191300
This is cleaning up all of that tech debt.
Changes
_api_key
is nowpassword
and we use the existing credentials helper.Contentious points
api_key
has changed topassword
, I can add a small helper to pull from this field ifpassword
is none etc but curious thoughts on that one.self._cookie
which is now holding the base64 encoded authorization bearer token.Testing
This was tested in conjunction with https://github.com/airbnb/streamalert/pull/1311 against a live environment.