Skip to content
This repository has been archived by the owner on Dec 20, 2022. It is now read-only.

[patch] [WIP] setcap in docker image #96

Closed
wants to merge 1 commit into from
Closed

[patch] [WIP] setcap in docker image #96

wants to merge 1 commit into from

Conversation

WindzCUHK
Copy link
Contributor

Description

setcap to allow binding well-known port for non-root user

Type of change

  • Bug fix
  • New feature
  • Refactoring (no functional changes, no api changes)
  • Non-code changes (update documentation, pipeline, etc.)

Flags

  • breaks backward compatibility
  • requires a documentation update
  • has untestable code

Related issue/PR

Checklist

  • Followed the guidelines in the CONTRIBUTING document
  • Added prefix [major]/[minor]/[patch]/[skip] in the PR title
  • Tested and linted the code
  • Commented the code
  • Made corresponding changes to the documentation
  • Confirmed no dropping in test coverage (by Codecov)
  • Passed all pipeline checking
  • Approved by >1 reviewer

Checklist for maintainer

  • Use Squash and merge
  • Double-confirm the merge message has prefix [major]/[minor]/[patch]/[skip]
  • Delete the branch after merge

@WindzCUHK WindzCUHK mentioned this pull request Dec 20, 2022
Open
16 tasks
@WindzCUHK
Copy link
Contributor Author

continue in: AthenZ/authorization-proxy#13

@WindzCUHK WindzCUHK closed this Dec 20, 2022
@WindzCUHK WindzCUHK deleted the setcap branch December 20, 2022 08:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@WindzCUHK