-
Notifications
You must be signed in to change notification settings - Fork 5
Merge release/lightspark-sdk-v0.20.1 into main #243
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
base: main
Are you sure you want to change the base?
Conversation
…18.0 Merge release/lightspark-sdk-v0.18.0 into develop
Add the uma-configuration file to the demo VASP
Initial commit - rename vasp1 & vasp 2 to sending/receiving vasp, add skeleton methods for new routes
# Receiver Vasp This PR adds an endpoint to create an invoice, and an endpoint to create invoice + send to another uma address # Testing run a receiver vasp on port 8080, and a sender vasp on 8081 to create an invoice `curl -u <username>:<password> -X POST "http://localhost:8080/api/uma/create_invoice?amount=100¤cyCode=USD"` to create and send an invoice to another uma `curl -u <username>:<password> -X POST "http://localhost:8080/api/uma/create_and_send_invoice/?amount=100¤cyCode=USD&senderUma=<sender uma>"`
# Sender VASP demo endpoints this adds 2 demo endpoints - receive an invoice from a receiver vasp and cache it for later client payment - process a payrequest for a full incoming uma invoice, or a cached invoice # testing run 2 vasps, a receiver vasp on 8080 and a sender vasp on 8081 receiver invoice to sender vasp `curl -u <username>:<password> -X POST "http://localhost:8081/api/uma/request_and_pay_invoice?invoice=<bech32 encoded invoice>"` sender client to sender vasp: `curl -u <username>:<password> -X POST "http://localhost:8081/api/uma/pay_invoice?invoice=<bech32 encoded invoice, or uuid identifying a cached invoice>`
Add explicit timeouts to the Requester
Create SECURITY.md
Regenerating SDKs following introspection of schemas.
This PR was automatically created by the release branch cut workflow.\nSDK=lightspark-sdk\nVERSION=0.19.0 Co-authored-by: runner <[email protected]>
This was accidental copy-pasta.
…19.1 Merge release/lightspark-sdk-v0.19.1 into develop
…urlInvoice, and createUmaInvoice and their respective mutations from all 3 clients.
…nt_has-and-preimage_nonce-parameters Added paymentHash and preimageNonce params to createInvoice functions
…leasepaymentpreimage-function Added releasePaymentPreimage functions to all 3 kotlin clients
Update Gradle version to 8.14.2
Fix misnamed buckets ldev.web-dev -> lsdev.web-dev
Merge release/core-v0.6.1 into develop
…20.0 Merge release/lightspark-sdk-v0.20.0 into develop
Improve webhook request validation and test coverage
import com.lightspark.sdk.core.LightsparkException | ||
import com.lightspark.sdk.model.WebhookEventType | ||
import com.lightspark.sdk.util.serializerFormat | ||
import java.security.MessageDigest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Critical compilation error: java.security.MessageDigest
is imported in commonMain code, but this is a JVM-only class. In Kotlin Multiplatform, commonMain source sets must only use APIs available on all targets (JVM, JS, Native). This will cause compilation failures on JavaScript and Native targets. The import needs to be moved to platform-specific source sets or replaced with a multiplatform-compatible constant-time comparison function.
Spotted by Diamond
Is this helpful? React 👍 or 👎 to let us know.
This PR was automatically created by the release branch cut workflow.\nSDK=lightspark-sdk\nVERSION=0.20.1