Skip to content

[Fix-11031] AccessToken can only be used by the creator.#11032

Merged
caishunfeng merged 5 commits intoapache:devfrom
WangJPLeo:access-token-no-permission
Jul 20, 2022
Merged

[Fix-11031] AccessToken can only be used by the creator.#11032
caishunfeng merged 5 commits intoapache:devfrom
WangJPLeo:access-token-no-permission

Conversation

@WangJPLeo
Copy link
Contributor

Purpose of the pull request

AccessToken can only be used by the creator.

Brief change log

close #11031

Verify this pull request

Manually verified the change by testing locally.

@codecov-commenter
Copy link

codecov-commenter commented Jul 18, 2022

Codecov Report

Merging #11032 (05cef6a) into dev (6bf80f7) will decrease coverage by 0.06%.
The diff coverage is 40.00%.

@@             Coverage Diff              @@
##                dev   #11032      +/-   ##
============================================
- Coverage     40.36%   40.29%   -0.07%     
+ Complexity     4865     4856       -9     
============================================
  Files           943      943              
  Lines         37061    37045      -16     
  Branches       4068     4067       -1     
============================================
- Hits          14959    14929      -30     
- Misses        20599    20610      +11     
- Partials       1503     1506       +3     
Impacted Files Coverage Δ
...duler/api/service/impl/AccessTokenServiceImpl.java 78.65% <40.00%> (-10.38%) ⬇️
...duler/plugin/task/zeppelin/ZeppelinParameters.java 70.00% <0.00%> (-11.25%) ⬇️
...heduler/plugin/task/jupyter/JupyterParameters.java 80.00% <0.00%> (-11.18%) ⬇️
...erver/master/processor/queue/TaskEventService.java 69.64% <0.00%> (-10.72%) ⬇️
...r/plugin/task/sqoop/parameter/SqoopParameters.java 53.33% <0.00%> (-1.34%) ⬇️
...permission/ResourcePermissionCheckServiceImpl.java 66.66% <0.00%> (-0.78%) ⬇️
...inscheduler/service/expand/CuringGlobalParams.java 33.33% <0.00%> (-0.39%) ⬇️
...rver/master/runner/task/BlockingTaskProcessor.java 75.60% <0.00%> (-0.30%) ⬇️
...eduler/api/service/impl/DataSourceServiceImpl.java 29.49% <0.00%> (ø)
...nscheduler/service/process/ProcessServiceImpl.java 29.29% <0.00%> (ø)
... and 6 more

@sonarqubecloud
Copy link

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

62.2% 62.2% Coverage
14.8% 14.8% Duplication

Copy link
Contributor

@caishunfeng caishunfeng left a comment

Choose a reason for hiding this comment

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

LGTM

@caishunfeng caishunfeng merged commit fd59f0b into apache:dev Jul 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature][AccessToken] AccessToken can only be used by the creator.

3 participants