-
Notifications
You must be signed in to change notification settings - Fork 19
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
[feat] Add token as authentication for python gateway #13
Conversation
separate from apache/dolphinscheduler#6407. Authentication, add secret to ensure only trusted people could connect to gateway. fix: apache/dolphinscheduler#8255
Codecov Report
@@ Coverage Diff @@
## main #13 +/- ##
==========================================
- Coverage 90.60% 90.59% -0.01%
==========================================
Files 56 56
Lines 2032 2042 +10
==========================================
+ Hits 1841 1850 +9
- Misses 191 192 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
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.
LGTM
released after apache/dolphinscheduler:3.1.2 |
Brief Summary of The Change
separate from apache/dolphinscheduler#6407. Authentication, add secret to ensure only trusted people could
connect to gateway.
fix: apache/dolphinscheduler#8255
ref: apache/dolphinscheduler#12893
Pull Request checklist
I confirm that the following checklist has been completed.
UPDATING.md
when it is an incompatible change.