search

dspinellis / git-issue

Git-based decentralized issue management
GNU General Public License v3.0
776 stars 66 forks source link

Fix issue#70 #71

Closed mjyc closed 4 years ago

mjyc commented 4 years ago

A quick fix based on findings from https://github.com/dspinellis/git-issue/issues/70#issuecomment-606142066

dspinellis commented 4 years ago

Is this covered by PR #73? Why are we changing the Link header match?

mjyc commented 4 years ago

Opps, I used a wrong comment reference, the correct one is at here https://github.com/dspinellis/git-issue/issues/70#issuecomment-607022134

I temporarily introduce Link header match because I didn't know about what existing '^\(Status: 200\|HTTP/[[:digit:]].[[:digit:]] 200 OK\)' match was doing, which I now think is a better rule to use.

mjyc commented 4 years ago

Also is not covered by #73; that PR is fixing an export related problem and this one is fixing an import related problem (but essentially the same cause)

dspinellis commented 4 years ago

Thank you for the explanation! To avoid a confusing history, can you please force push a patch without the intermediate Link header match?

mjyc commented 4 years ago

Modified commit history as requested

dspinellis commented 4 years ago

Thank you!