Skip to content
This repository has been archived by the owner on May 3, 2020. It is now read-only.

Merge changes from upstream #6

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

Conversation

ibnesayeed
Copy link

Checking for the possibility of merging recent changes from the upstream while still utilizing the multi-handler support that was not merged in the upstream.

/cc @jirikuncar

@jirikuncar
Copy link

@ibnesayeed I have updated the PR with multi-handler support in upstream. Can we close the PR?

@ibnesayeed
Copy link
Author

Thanks for your help @jirikuncar. I think the upstream repo is not planning to accept the multi-handler support, so I thought I will use this fork instead. However, some new changes in the upstream repo like addition of user-agent header (mementoweb#25) and removal of stale TameGates are important that this fork may benefit from. The purpose of this PR was to bring it to your attention. Please feel free to close it if you are not planning to merge those new changes from the upstream or prefer doing them differently.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants