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)
Mon, May 20, 9:34 PM
Unknown Object (File)
Mon, May 20, 4:31 PM
Unknown Object (File)
Mon, May 20, 1:25 PM
Unknown Object (File)
Sun, May 19, 10:53 AM
Unknown Object (File)
Tue, May 7, 2:56 PM
Unknown Object (File)
Sun, May 5, 7:44 PM
Unknown Object (File)
Sun, May 5, 3:43 PM
Unknown Object (File)
Thu, Apr 25, 8:13 AM
View All 48 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...