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

[release staging] April 2021 Release (v13.6.0) #1650

Closed
wants to merge 46 commits into from
Closed

Conversation

profnandaa
Copy link
Member

13.6.0 - 21/04 Release

profnandaa and others added 30 commits November 30, 2020 16:51
fix failure due to missing files
we recalled 13.5.0 due to build issues #1538 #1537
* feat(isVAT): add Italian regex

* test(isVAT): add Italian regex testing
* feat Add support for Angola mobile phone numbers

* feat: update readme
* fix(isMobilePhone): update de-CH locale (#1549)
feat(isMobilePhone): add fr-CH locale (#1549)
feat(isMobilePhone): add it-CH locale (#1549)

* fix(isMobilePhone): update de-CH, fr-CH and it-CH locale in validator.min.js(#1549)

* fix(isMobilePhone): update fr-CH and it-CH locale aliases (#1549)

Co-authored-by: Ashutosh Kumar <[email protected]>
* Added isLicensePlate check with german locale

* de-LI (Liechtenstein) validator

* Added albanian number plates

* reversed changes in index.js

* reversed changes in index.js

* Refactored de-DE into one big regex, expanded tests fot de-DE

* added pt-PT locale

* Fixed coverage problem

* feat(isLicensePlate): clean build

* coverage is now 100%

* Update README.md

* Update README.md
Add correct tax identifier to demonstrate issue and fix.

Thanks to: Panos Papadopoulos
)

* Support new VN mobile phone 087 (https://didong.itelecom.vn/)

* Add test for new telco number

Co-authored-by: Kyle Dinh <[email protected]>
* chore: prevent git from ignoring src/index.js file

* chore: remove unused exclusion from nyc config

* chore: replace travis-ci with github actions

* chore: fix issue with github workflow config

* chore: fix condition for coverage generation and sending

* chore: treat node version as int

* docs: replace travis with github actions badge

* chore: add npm publish github action

* chore: remove auto-generated files

* chore: improve github actions styles

* chore: use correct repo url to prevent unecessary redirect

* chore: lint package.json file

* chore: add new line at end of file

* chore: set a fixed ubuntu version instead of relying on latest
* refactor code in isdatauri validator

* Delete validator.js

* Delete validator.min.js

Co-authored-by: Sarhan Aissi <[email protected]>
* fix(isMacAddress): improve Regexes

* docs: update docs

* Update src/lib/isMACAddress.js

* Update src/lib/isMACAddress.js

* Update test/validators.js

* Update README.md
* Added Libya Passort and Identity Number Validation

* Delete validator.js

* Delete validator.min.js

Co-authored-by: Sarhan Aissi <[email protected]>
* test: add `isSlug` test cases

* fix: remove duplicate test case
Change Text from "Passing anything other than a string is an error." to "Passing anything other than a string will result in an error."
… update (#1577)

* added support for EAN-14 along with test case

* feat(isAlphanumeric): added options(optional parameter)

* Delete validator.min.js

* Delete validator.js

Co-authored-by: Sarhan Aissi <[email protected]>
* Improving code coverage to 100% branches

* Improving code coverage for isStrongPassword
* fix(isMobilePhone): prevent validator from allowing landline numbers in es-CO locale

* fix(isMobilePhone): prevent es-CO validator from allowing invalid number
* feat(pt-BR): tax id, passport and license plates

* delete compiled files

* fix(tests): tests now covering 100% of code

* fix(licenseplate): removed license plate pt-br

* fix(readme): removed license plate changes
* License Plate validators: pt-BR and mercosur

* Removing mercosur as it's not an actual valid country locale
* [feat] Malaysian Passport validation

* Delete validator.js

* Delete validator.min.js

Co-authored-by: Sarhan Aissi <[email protected]>
mhf-ir and others added 14 commits April 17, 2021 16:10
* feat: isLuhn, IR passport and identityCard, respect .gitignore files

* Update test/validators.js

Co-authored-by: Federico Ciardi <[email protected]>

* fix: rEADME.md

* fix: remove isLuhn

Co-authored-by: Federico Ciardi <[email protected]>
A number of files contain ranges of [A-z] which allows the characters [\]^_\ and the back-tick character. Those have been replaced by [A-Za-z].
* fix: 🐛 ipv4/ipv6 validators

Provide a more strict regular expression patterns for ipv4 and ipv6
formats.

✅ Closes: #1626

* test: 💍 ipv4/ipv6 unit tests

Extend upon existing IP validator unit test, valid examples. Provide
IPv6 test addresses including `%`.

✅ Closes: #1626
* optimize isISIN speed + gc

* add aapl ISIN

* comment and reference
…ent (#1644)

* Added some missing options to the isURL() docs

* Allow URLs to have a userinfo section with only a username

The 'userinfo' part of a URL may, according to RFC 1738, contain only
a username followed by an '@' sign. The previous behavior of the
isURL() function would return false if the userinfo section did not
have a colon.

In addition to the change in the function, tests have been added to
ensure the following exmaples are considered valid:

 - http://[email protected]
 - http://user:@example.com
 - http://user:[email protected]

The following are considered not valid:

 - http://@example.com
 - http://:@example.com
 - http://:example.com

As a practical example, Sentry (https://github.com/getsentry/sentry)
uses a format like http://[email protected]/10
for it's DSNs (which are just URLs).
…1604)

* added instruction into isPassportNumber to validate Mozambican passport number.

* Add to (isMobilePhone) to validate numbers from Mozambique

* Deleted validator.js and validator.min.js

* docs: remove trailing space from isPassportNumber

* isIBAM Mozambique

Co-authored-by: Sarhan Aissi <[email protected]>
@profnandaa
Copy link
Member Author

@tux-tn @ezkemboi @chriso -- I've prepared the release and ready to publish to NPM. Just wanted you folks have a look and give your stamp.

Copy link
Member

@tux-tn tux-tn left a comment

Choose a reason for hiding this comment

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

LGTM !
I'm hoping we can include #1651 in the release even if it was shipped late

* chore: bump mocha version to fix npm audit warning

* fix(isHSL): update hslComma regex to prevent ReDOS

* fix(isEmail): update splitNameAddress regex to prevent ReDOS

* chore: rollback mocha version to allow testing on node 8 and 6

* fix(isHSL): remove unnecessary use of let

closes #1597 #1598
@codecov
Copy link

codecov bot commented Apr 20, 2021

Codecov Report

Merging #1650 (24b3fd3) into v13.5.1 (012301d) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##           v13.5.1     #1650   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           99       101    +2     
  Lines         1773      1854   +81     
=========================================
+ Hits          1773      1854   +81     
Impacted Files Coverage Δ
src/lib/isIBAN.js 100.00% <ø> (ø)
src/lib/isPassportNumber.js 100.00% <ø> (ø)
src/lib/isPostalCode.js 100.00% <ø> (ø)
src/lib/isVAT.js 100.00% <ø> (ø)
src/index.js 100.00% <100.00%> (ø)
src/lib/isAlphanumeric.js 100.00% <100.00%> (ø)
src/lib/isBIC.js 100.00% <100.00%> (ø)
src/lib/isBtcAddress.js 100.00% <100.00%> (ø)
src/lib/isDataURI.js 100.00% <100.00%> (ø)
src/lib/isEAN.js 100.00% <100.00%> (ø)
... and 21 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 012301d...24b3fd3. Read the comment docs.

@profnandaa
Copy link
Member Author

Released https://www.npmjs.com/package/validator/v/13.6.0 (includes the ReDOS fixes).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.