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

payAck.php returns wrong Content-Length to client #8

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

DanielWeigl
Copy link

Wrong calculation of the content-length in payAck.php - depending on the http-client behavior, it might get a truncated response.

@gavinandresen
Copy link
Owner

Thanks! If I forget to pull this and update the code running on bitcoincore.org within the next week, please remind me.

@DanielWeigl
Copy link
Author

Please also check #7 - I guess the cert-files got moved or are somehow not accessible by the script.

@DanielWeigl
Copy link
Author

Wow, this is an old one :) - already forgotten about it...

Is this demo script still somewhere running? the original URL ( https://bitcoincore.org/~gavin/createpaymentrequest.php ) is a 404 (which is even mentioned in the BIP70 and many other locations)

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.

3 participants