Skip to content
This repository has been archived by the owner on Apr 15, 2024. It is now read-only.

fix: skip tls verify while working on Wheatley replacement #1

Merged
merged 1 commit into from Mar 30, 2023
Merged

fix: skip tls verify while working on Wheatley replacement #1

merged 1 commit into from Mar 30, 2023

Conversation

chatchai-outreach
Copy link

@chatchai-outreach chatchai-outreach commented Mar 30, 2023 •
edited

  • upgrade faye-websocket for tls verify_peer
  • set verify_peer to false to skip tls verify

https://outreach-io.atlassian.net/browse/DT-3634

@chatchai-outreach chatchai-outreach requested a review from a team March 30, 2023 16:32
@swarmia
Copy link

swarmia bot commented Mar 30, 2023

✅  Linked to DT-2936 · Find a long term solution for Wheatley

RhymePurging
RhymePurging approved these changes Mar 30, 2023
View reviewed changes
Copy link

@RhymePurging RhymePurging left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - skip tls verify while we migrate wheatley

@chatchai-outreach chatchai-outreach merged commit 410efe1 into master Mar 30, 2023
@chatchai-outreach chatchai-outreach deleted the cluangma/fix/skip-verify-tls branch March 30, 2023 17:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@RhymePurging RhymePurging RhymePurging approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@chatchai-outreach @RhymePurging