Skip to content

Fixes inspired by KYOCERA ECOSYS M2540dn#222

Closed
kpnr wants to merge 7 commits intoctalkington:masterfrom
kpnr:master
Closed

Fixes inspired by KYOCERA ECOSYS M2540dn#222
kpnr wants to merge 7 commits intoctalkington:masterfrom
kpnr:master

Conversation

@kpnr
Copy link
Copy Markdown

@kpnr kpnr commented Sep 30, 2021

Good day, Chris!

After some requests to IPP interface in KYOCERA ECOSYS M2540dn found some minor bugs/incompleteness.

Fixes:

  • "successful" & "informatinal" status-codes not raise exception now
  • message merge bug fix

Improvements:

  • parsing "unsupported-attribues" group
  • parsing "DATE" attributes
  • added two tags to ATTRIBUTE_TAG_MAP

All changes are covered with tests.

Awaiting your review & remarks.

Thank you, hkm...

- Added DATE parsing into Python datetime object
- Added "unsupported-attributes" parsing
trailing space removed
Added "ipp-attribute-fidelity" & "orientation-requested"
- status code in "successful" & "informational" range (0x0...0x1FF) not raising
- fixed message merging (IPP._message)
"unsupported-attributes" and "informational status code" blobs
Test case for DATE, informational status-code
message merge & non-zero status-code tests
@ctalkington
Copy link
Copy Markdown
Owner

ctalkington commented Oct 6, 2022

thanks for these updates. sorry it took some time to get back to this project.

I had to manually re-implement them due to other big changes but I added you to the attribution in v0.12.0 release notes.

@ctalkington ctalkington closed this Oct 6, 2022
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.

2 participants