Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Duplicated printer-device-id attribute breaks get-printer-attributes response #23

Open
zdohnal opened this issue Nov 1, 2023 · 2 comments

Comments

@zdohnal
Copy link
Member

zdohnal commented Nov 1, 2023

Hi,

I've installed a printer via snapped ps-printer-app and tried to query it via ipptool for available attributes:

$ ipptool -tv ipp://localhost:8000/ipp/print/mytest get-printer-attributes.test
"/usr/share/cups/ipptool/get-printer-attributes.test":
    Get-Printer-Attributes:
        attributes-charset (charset) = utf-8
        attributes-natural-language (naturalLanguage) = en
        printer-uri (uri) = ipp://localhost:8000/ipp/print/mytest
        requested-attributes (1setOf keyword) = all,media-col-database
    Get printer attributes using get-printer-attributes                  [FAIL]
        RECEIVED: 15164 bytes in response
...
        Duplicate "printer-device-id" attribute in printer-attributes-tag group
$ ipptool -tv ipp://localhost:8000/ipp/print/mytest get-printer-attributes.test | grep printer-device-id
       printer-device-id (textWithoutLanguage) = MFG:Hewlett-Packard;CMD:PJL,PML,PCLXL,URP,PCL,PDF,POSTSCRIPT;MDL:HP LaserJet M1536dnf MFP;CLS:PRINTER;DES:Hewlett-Packard LaserJet M1536dnf MFP;MEM:MEM=112MB;COMMENT:RES=600x2;LEDMDIS:USB#ff#04#01;U
        printer-device-id (textWithoutLanguage) = MFG:Hewlett-Packard;CMD:PJL,PML,PCLXL,URP,PCL,PDF,POSTSCRIPT;MDL:HP LaserJet M1536dnf MFP;CLS:PRINTER;DES:Hewlett-Packard LaserJet M1536dnf MFP;MEM:MEM=112MB;COMMENT:RES=600x2;LEDMDIS:USB#ff#04#01;U
    Duplicate "printer-device-id" attribute in printer-attributes-tag group

I was not able to debug further at the moment, I will let you know if I find something.

@tangyanli
Copy link

The issue has been fixed at pappl side, related issue: michaelrsweet/pappl#328

@tillkamppeter
Copy link
Member

@zdohnal could you test again to ee whether it is actually fixed? Should be so ...

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

No branches or pull requests

3 participants