search

zopefoundation / Products.PluggableAuthService

Pluggable Zope authentication / authorization framework
Other
9 stars 18 forks source link

Zope 4 fixes. #4

Closed jensens closed 7 years ago

jensens commented 7 years ago

Fixes made in order to work with Zope 4, includes some common findings.

icemac commented 7 years ago

It would be nice to see if these changes pass on Travis, see #5 which still requires Travis CI to be switched on for this repository (#6).

icemac commented 7 years ago

Travis is enabled now, could you please rebase your PR to the current master?

jensens commented 7 years ago

rebased

tests are red, seems like a returned 302 signature and actual headers are different (so not wrong afaict).

icemac commented 7 years ago

Now after #7 is merged to master this PR could be rebased to the master.

icemac commented 7 years ago

@jensens Can I safely delete the branch – as I merged it to master – or will this break some integration tests?

icemac commented 7 years ago

I decided to delete the branch to signal that the integration tests now should use the master branch.