Skip to content

feat: expose printer-discovery-enabled via IPP attribute - #34

Open
liuli0217 wants to merge 1 commit into
deepin-community:masterfrom
liuli0217:master
Open

liuli0217 wants to merge 1 commit into
deepin-community:masterfrom
liuli0217:master

Conversation

@liuli0217

Copy link
Copy Markdown
Contributor

Parse the DisableAvahiBrowsing directive in cupsd and expose the printer discovery setting via the public IPP attribute printer-discovery-enabled so clients can read it without admin authentication, including inside containers. fixed for linglong.

Parse the DisableAvahiBrowsing directive in cupsd and expose the printer
discovery setting via the public IPP attribute printer-discovery-enabled
so clients can read it without admin authentication, including inside
containers.

cupsEnumDests() now queries CUPS-Get-Default instead of parsing cupsd.conf
directly, which failed for unprivileged or containerized clients that
cannot read the scheduler configuration.

Log: 通过 IPP 属性暴露并读取网络发现开关
Influence: 容器内客户端无需管理员认证即可正确读取网络发现配置。
Change-Id: Ia36691dd864458ef8606f1e84418abe65a997b3b
@deepin-ci-robot

Copy link
Copy Markdown
Contributor

/hold
因为该quilt包的上游版本号变更,详情见: deepin-community/infra-settings#134

@github-actions

github-actions Bot commented Nov 5, 2026

Copy link
Copy Markdown

TAG Bot

TAG: 2.4.16-1deepin22
EXISTED: no
DISTRIBUTION: unstable

@deepin-ci-robot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign tsic404 for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

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