Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Allow authorise-without-apparmor when aa_getpeercon() returns ENOPROTOOPT #175

Merged
merged 2 commits into from
Apr 12, 2024
Merged

Allow authorise-without-apparmor when aa_getpeercon() returns ENOPROTOOPT #175

merged 2 commits into from
Apr 12, 2024

Conversation

AlanGriffiths
Copy link
Contributor

@AlanGriffiths AlanGriffiths commented Apr 5, 2024 •
edited
Loading

Fixes: #167

@AlanGriffiths AlanGriffiths marked this pull request as ready for review April 12, 2024 08:38
@AlanGriffiths AlanGriffiths requested a review from a team as a code owner April 12, 2024 08:38
@AlanGriffiths
Copy link
Contributor Author

Confirmed working on Debian/buster

Saviq
Saviq requested changes Apr 12, 2024
View reviewed changes
src/frame_main.cpp Outdated Show resolved Hide resolved
src/snap_name_of.cpp Outdated Show resolved Hide resolved
@AlanGriffiths @Saviq
Apply suggestions from code review
ef59c81 
Co-authored-by: Michał Sawicz <michal.sawicz@canonical.com>
@AlanGriffiths AlanGriffiths requested a review from Saviq April 12, 2024 10:02
Saviq
Saviq approved these changes Apr 12, 2024
View reviewed changes
Copy link
Collaborator

@Saviq Saviq left a comment

Choose a reason for hiding this comment

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

Yups.

@Saviq Saviq added this pull request to the merge queue Apr 12, 2024
Merged via the queue into main with commit 9643015 Apr 12, 2024
3 checks passed
@Saviq Saviq deleted the incompatible-AppArmor branch April 12, 2024 10:22
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@Saviq Saviq Saviq approved these changes

Assignees
No one assigned
Labels
needs test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

aa_getpeercon() failed for process: Protocol not available
2 participants
@AlanGriffiths @Saviq