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

fix: properly encode the LNURLp payerdata JSON #1715

Merged
merged 3 commits into from
Nov 9, 2022
Merged

Conversation

bumi
Copy link
Collaborator

@bumi bumi commented Nov 7, 2022

Maybe this worked because the old axios version did that automatically (which was wrong)

Describe the changes you have made in this PR

I think the updated axios version #1702 behaves differently.
We had passed in the query params here wrong and did not encode the JSON manually.
Thus the payerdata was sent wrong to the server

Type of change

(Remove other not matching type)

  • fix: Bug fix (non-breaking change which fixes an issue)

How has this been tested?

send a payment to a LN address that supports payerdata.

Maybe this worked because the old axios version did that automatically (which was wrong)
@bumi bumi requested a review from escapedcat November 7, 2022 08:15
@github-actions
Copy link

github-actions bot commented Nov 7, 2022

🚀 Thanks for the pull request!

Here are the current build files for testing:

Download and unzip the file for your browser. Refer to the readme for detailed install instructions.


This build is brought to you by: Adam Fiscor (who recently dropped 1337 sats):

The future is bright. We just have a lot of work to do.

Want to sponsor the next build? send some sats to ⚡️builds@getalby.com (don't forget to provide your name)

Don't forget: keep earning sats!

bumi added 2 commits November 9, 2022 16:55
* master: (21 commits)
  chore: fix lodash types and reduce package size
  chore: remove storybook github action
  chore: remove storybook stories
  chore: remove storybook config
  docs: adjust storybook info
  chore: remove storybook
  Translated using Weblate (Italian)
  fix: handle dexie in jest
  Update react-router-dom to version 6.4.3
  Update i18next-browser-languagedetector to version 7.0.1
  Update react-toastify to version 9.1.1
  fix: yarn lock
  fix: use transformIgnorePatterns for dexie
  chore: remove unused runcitadel sdk
  prettier i18nConfig.ts
  feat(i18n): activate italian
  feat: handle organization pages
  fix: highlight menu icon dots
  fix: tipping in medium
  fix: fetch from description element
  ...
@bumi bumi merged commit f73196e into master Nov 9, 2022
@bumi bumi deleted the fix/lnurlp-request branch November 9, 2022 16:03
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.

2 participants