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

Add domain validation during ImgixClient initialization #37

Merged
merged 3 commits into from
Mar 7, 2019

Conversation

sherwinski
Copy link
Contributor

Adds a validation that will throw an error if a user attempts to initialize a new ImgixClient object with an invalid domain.
The validation criteria is based off of the same regex used on dashboard.imgix.com

@sherwinski sherwinski force-pushed the domain-validation branch 2 times, most recently from 6944323 to d400994 Compare February 27, 2019 23:40
Copy link
Contributor

@jayeb jayeb left a comment

Choose a reason for hiding this comment

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

A couple of codestyle nits, but the changes here look good. As a procedural note, we try not to check in dist/* files into branches--those should be built and commited to master during the versioning step of the release process.

src/imgix-core-js.js Outdated Show resolved Hide resolved
src/imgix-core-js.js Show resolved Hide resolved
@sherwinski sherwinski requested review from jayeb and removed request for jayeb March 7, 2019 22:27
@sherwinski sherwinski merged commit 540ecfa into master Mar 7, 2019
@sherwinski sherwinski deleted the domain-validation branch March 7, 2019 23:17
@ericdeansanchez
Copy link
Contributor

🎉 This PR is included in version 0.3.0-beta.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

None yet

3 participants