Skip to content

[Filebrowser] Add get delegation token logic for secure hadoop (#3301) (Related with: #3324,#3449 ) #3814

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

Open
wants to merge 21 commits into
base: master
Choose a base branch
from

Conversation

SoniaComp
Copy link
Contributor

What changes were proposed in this pull request?

I am working on integrating Hue with SecureHadoop as a ProxyUser.
I found that the behavior of ProxyUser in WebHDFS FileBrowser needs to be enhanced.
When "security_enabled" is true, only "read_url" method use delegation token.

I want to modify the code
so that a user who has been impersonated from Hue(a proxy user) can issue a Delegation Token when using WebHDFS.
(#3323)

How was this patch tested?

Our company has been successfully using this fixed code since last year, so I'm sure that this bug fix work well.

Copy link

This PR is stale because it has been open 45 days with no activity and is not labeled "Prevent stale". Remove "stale" label or comment or this will be closed in 10 days.

@github-actions github-actions bot added the Stale label Sep 23, 2024
@bjornalm bjornalm requested a review from agl29 September 23, 2024 12:09
@bjornalm
Copy link
Collaborator

@SoniaComp Thank you for contributing with this PR.

@agl29 Can you take a look at this? It has been pending for a while.

Copy link

github-actions bot commented Jun 4, 2025

⚠️ No test files modified. Please ensure that changes are properly tested. ⚠️

Copy link

github-actions bot commented Jun 4, 2025

UI Coverage Report

Lines Statements Branches Functions
Coverage: 32%
39.15% (30527/77959) 31.01% (14247/45936) 23.89% (2130/8915)

Copy link

github-actions bot commented Jun 4, 2025

Python Code Coverage

Python Coverage Report •
FileStmtsMissCoverMissing
desktop/libs/hadoop/src/hadoop/fs
   webhdfs.py66546530%126, 134, 136, 144, 148, 152, 156, 160–162, 164, 166–170, 172, 182–189, 191–196, 198–199, 202, 220–221, 232, 240, 244–246, 248–251, 259–267, 275–276, 282–287, 297, 300, 307–310, 316–319, 322–325, 328, 332–334, 345–347, 349–350, 352–354, 357–360, 363–364, 372–377, 380–381, 385–386, 388, 392, 396–397, 399, 409–410, 413–419, 423–425, 433–434, 442–448, 450–452, 456–461, 463–467, 471–479, 483–488, 492–502, 504, 512–519, 521, 529–531, 534, 542–553, 556–561, 569–582, 592, 595, 598, 607–619, 627–631, 635–640, 643–648, 651–655, 658–662, 665–670, 673–677, 680–690, 692, 695, 698–704, 706, 708–709, 711–712, 714, 716–721, 728, 730–731, 733, 736–737, 739–740, 742–743, 745–749, 751, 770–771, 776–778, 780–781, 783–784, 786, 789–791, 796–798, 800, 802, 805, 810–811, 813, 820, 829–830, 832–833, 836, 838–839, 842, 845–849, 853, 855–857, 859–865, 870–871, 873–878, 889, 892–898, 902–904, 906–907, 909, 912, 914, 916–917, 931–938, 940–947, 949, 953–959, 961, 964–965, 968, 971–973, 977, 980–982, 985, 988, 997–1000, 1008–1011, 1018–1019, 1021–1022, 1025–1031, 1034–1039, 1042–1047, 1051–1055, 1057
TOTAL541852707850% 

Pytest Report

Tests Skipped Failures Errors Time
1186 106 💤 0 ❌ 0 🔥 6m 2s ⏱️

@SoniaComp
Copy link
Contributor Author

@bjornalm @agl29 Hi! Could I get more feedbacks about this PR? I could develop the code than! Thank you 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants