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

Assorted fixes/improvements #178

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Assorted fixes/improvements #178

wants to merge 7 commits into from

Conversation

PaulSD
Copy link

@PaulSD PaulSD commented Oct 9, 2019

Back in 2016, I was working on a series of patches to mod_auth_cas to fix several bugs and add various features. I made some significant progress, and we started using most of my patches in our environment, but I figured I would wait until I had the whole series finished before I submitted a PR. Unfortunately, I got pulled off to another project before that happened, and my patch series was forgotten ... until we decided to upgrade to mod_auth_cas v1.2 and re-discovered this old work. Some of my old changes have already been implemented in v1.2 by other people, some of my old patches have merge conflicts with v1.2 that I haven't resolved yet, and some of my old patches are still incomplete and not ready for merging upstream ... But several patches are ready for a PR, so I figured I should open one for them before I get busy with something else and they get forgotten again.

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.

1 participant