-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
fix(providers): add SSL renegotiation for Gupshup #5607
base: next
Are you sure you want to change the base?
fix(providers): add SSL renegotiation for Gupshup #5607
Conversation
✅ Deploy Preview for dev-web-novu ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for novu-design ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@RahulDutta007 Thanks for the contribution. We will review it, first thing in the morning. |
const response = await axios.post(GupshupSmsProvider.BASE_URL, params); | ||
const response = await axios.post(GupshupSmsProvider.BASE_URL, params, { | ||
httpsAgent: new Agent({ | ||
rejectUnauthorized: false, |
Check failure
Code scanning / CodeQL
Disabling certificate validation High
What changed? Why was the change needed?
Enable legacy SSL renegotiation for Gupshup endpoint
Screenshots
Related enterprise PR
Special notes for your reviewer
@SokratisVidros any chance this can be added a hotfix to current docker image in ghcr.io. We are using this in our organization