Skip to content

Deprecate auth package and TNetFile#21775

Open
jblomer wants to merge 30 commits intoroot-project:masterfrom
jblomer:deprecate-rootd
Open

Deprecate auth package and TNetFile#21775
jblomer wants to merge 30 commits intoroot-project:masterfrom
jblomer:deprecate-rootd

Conversation

@jblomer
Copy link
Copy Markdown
Contributor

@jblomer jblomer commented Apr 2, 2026

Deprecates TVirtualAuth, the concrete implementation (auth package), and dependencies:

  • Socket authentication
  • TROOT::GetListOfSecContexts
  • TNetFile/TNetFileStager/TNetFileSystem
  • TFTP

@jblomer jblomer self-assigned this Apr 2, 2026
@jblomer jblomer changed the title Deprecate auth package and dependencies Deprecate auth package and TNetFile Apr 2, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 2, 2026

Test Results

    22 files      22 suites   3d 3h 3m 27s ⏱️
 3 830 tests  3 829 ✅  1 💤 0 ❌
75 611 runs  75 593 ✅ 18 💤 0 ❌

Results for commit 548d6f7.

♻️ This comment has been updated with latest results.

Copy link
Copy Markdown
Member

@dpiparo dpiparo left a comment

Choose a reason for hiding this comment

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

Thanks. LGTM.

@dpiparo
Copy link
Copy Markdown
Member

dpiparo commented Apr 6, 2026

Maybe a small suggestion could be to update the documentation of sockets with something along the lines of 8cbf7e4 .

@hahnjo
Copy link
Copy Markdown
Member

hahnjo commented Apr 6, 2026

Regarding rpdutils, see 46eb07a. I think we cannot remove it (yet) without breaking the deprecated functionality.

@jblomer jblomer added the clean build Ask CI to do non-incremental build on PR label Apr 8, 2026
@jblomer jblomer closed this Apr 8, 2026
@jblomer jblomer reopened this Apr 8, 2026
@jblomer jblomer force-pushed the deprecate-rootd branch 2 times, most recently from 6eb75e3 to 548d6f7 Compare April 9, 2026 20:46
@jblomer jblomer requested review from dpiparo and silverweed April 10, 2026 06:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clean build Ask CI to do non-incremental build on PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants