Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@CodeWithBishal
Copy link
Contributor

@CodeWithBishal CodeWithBishal commented Jul 4, 2024

added a javascript file containing 3 functions. Each of the function represent each of the crypto addresses.

TO Use

include the javascript file, call the respective function with the address and the function will return true if it is a valid crypto address

TO Test

I have included a list of some demo crypto addresses to test each of the validator, TO Test: uncomment the respective validator and open the browser console on the page where the js file is included.

I will integrate this code with the profile page once, the profile page is completed
Integration with the existing profile page is completed ☑️

Thanks

this closes #2282 and #2281

@CodeWithBishal CodeWithBishal changed the title Bitcoin address validation using JavaScript Crypto address validation using JavaScript Jul 4, 2024
Copy link
Collaborator

@DonnieBLT DonnieBLT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good, can you please also integrate it into the site where the user enters addresses - on the profile page and on the bid page

@CodeWithBishal
Copy link
Contributor Author

Sure!

@CodeWithBishal
Copy link
Contributor Author

CodeWithBishal commented Jul 4, 2024

@DonnieBLT hey, so I found that user's crypto address is being updated only on the profile page, so I added the validator here,

I have checked bidding.html and bid_selection.html but couldn't find any forms updating the user's crypto address

this closes #2281

@CodeWithBishal CodeWithBishal enabled auto-merge July 5, 2024 16:52
can use this function to validate btc, eth and bch address while user is trying to
@CodeWithBishal CodeWithBishal merged commit 8aa28e9 into OWASP-BLT:main Jul 6, 2024
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.

Refine the validation logic based on test cases.

2 participants