Skip to content

feat: add bluetooth services to bluetooth toggle#1700

Draft
RoyalOughtness wants to merge 5 commits intosecureblue:livefrom
RoyalOughtness:obexd
Draft

feat: add bluetooth services to bluetooth toggle#1700
RoyalOughtness wants to merge 5 commits intosecureblue:livefrom
RoyalOughtness:obexd

Conversation

@RoyalOughtness
Copy link
Copy Markdown
Collaborator

No description provided.

@RoyalOughtness RoyalOughtness marked this pull request as ready for review January 12, 2026 20:07
Comment on lines +18 to +22
systemctl disable --user obex.service
systemctl mask --user obex.service

systemctl disable bluetooth.service
systemctl mask bluetooth.service No newline at end of file
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

These two services should also be preset-disabled.

enable secureblue-key-enrollment-verification.timer
enable secureblue-update-verification.timer
enable security-update-notification.path
enable security-update-notification.path No newline at end of file
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Unnecessary whitespace change.

Comment on lines +48 to +49
obex_service.unmask()
obex_service.enable_now()
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This is a user service and the inner script is running as root; wouldn't this either need to be done in the outer script (to operate on the service for the individual user) or use --global rather than --user to operate on the user service globally?

Comment on lines +56 to +57
obex_service.mask()
obex_service.disable_now()
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Likewise here.

@RoyalOughtness RoyalOughtness marked this pull request as draft March 9, 2026 18:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants