Page MenuHomePhorge
Differential D895

encode_page_control() must have a 2nd argument T5734
ClosedPublic
Actions

Authored by dhoffend on Dec 22 2019, 9:53 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Oct 27, 2:58 PM
Unknown Object (File)
Thu, Oct 10, 9:14 PM
Unknown Object (File)
Thu, Oct 10, 9:13 PM
Unknown Object (File)
Thu, Oct 10, 8:54 PM
Unknown Object (File)
Sep 11 2024, 6:50 AM
Unknown Object (File)
Sep 7 2024, 6:17 AM
Unknown Object (File)
Sep 7 2024, 4:22 AM
Unknown Object (File)
Sep 7 2024, 3:33 AM
View All 95 Files
Subscribers
None

Details

Reviewers
vanmeeuwen
Group Reviewers
PyKolab Developers
Commits
rP0e66fae6ac8c: encode_page_control() must have a 2nd argument T5734
Summary

As reported in T5734 kolab sync and kolabd are crashing
with a trackback that encode_page_control must have 2 arguments. Adding
the empty cookie attribute to the server_page_controls removed the error.

Diff Detail

Repository
rP pykolab
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dhoffend requested review of this revision.Dec 22 2019, 9:53 PM
dhoffend created this revision.
Harbormaster completed remote builds in B27547: Diff 2038.Dec 22 2019, 9:53 PM
dhoffend updated this revision to Diff 2044.Dec 23 2019, 6:30 PM
  • fix check for page control result
Harbormaster completed remote builds in B27553: Diff 2044.Dec 23 2019, 6:30 PM
dhoffend added a comment.Dec 23 2019, 6:31 PM

Hope this is correct from the documentation and examples about ldap paging search

vanmeeuwen accepted this revision.Jan 13 2020, 2:06 PM
This revision is now accepted and ready to land.Jan 13 2020, 2:06 PM
Closed by commit rP0e66fae6ac8c: encode_page_control() must have a 2nd argument T5734 (authored by dhoffend, committed by vanmeeuwen). · Explain WhyJan 13 2020, 2:12 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
pykolab/
auth/
ldap/
4 lines

Diff 2068

View Options

pykolab/auth/ldap/__init__.py

Loading...