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.

Sign up for GitHub

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

Jump to bottom

TypeError: Cannot supply flags when constructing one RegExp from another #3207

Closed
jannispl opened this issue Jul 25, 2016 · 3 comments
Closed

TypeError: Cannot supply flags when constructing one RegExp from another #3207

jannispl opened this issue Jul 25, 2016 · 3 comments
Labels
bug

Comments

@jannispl
Copy link 

TypeError: Cannot supply flags when constructing one RegExp from another
    at new RegExp (native)
    at /app/node_modules/keystone/lib/core/mount.js:228:44
    at Layer.handle [as handle_request] (/app/node_modules/express/lib/router/layer.js:95:5)
    at next (/app/node_modules/express/lib/router/route.js:131:13)
    at next (/app/node_modules/express/lib/router/route.js:125:14)
    at next (/app/node_modules/express/lib/router/route.js:125:14)
    at next (/app/node_modules/express/lib/router/route.js:125:14)
    at next (/app/node_modules/express/lib/router/route.js:125:14)
    at Route.dispatch (/app/node_modules/express/lib/router/route.js:112:3)
    at Layer.handle [as handle_request] (/app/node_modules/express/lib/router/layer.js:95:5)
    at /app/node_modules/express/lib/router/index.js:277:22
    at param (/app/node_modules/express/lib/router/index.js:349:14)
    at param (/app/node_modules/express/lib/router/index.js:365:14)
    at Function.process_params (/app/node_modules/express/lib/router/index.js:410:3)
    at next (/app/node_modules/express/lib/router/index.js:271:10)
    at expressInit (/app/node_modules/express/lib/middleware/init.js:33:5)
    at Layer.handle [as handle_request] (/app/node_modules/express/lib/router/layer.js:95:5)
    at trim_prefix (/app/node_modules/express/lib/router/index.js:312:13)
    at /app/node_modules/express/lib/router/index.js:280:7
    at Function.process_params (/app/node_modules/express/lib/router/index.js:330:12)
    at next (/app/node_modules/express/lib/router/index.js:271:10)
    at query (/app/node_modules/express/lib/middleware/query.js:44:5)
    at Layer.handle [as handle_request] (/app/node_modules/express/lib/router/layer.js:95:5)
    at trim_prefix (/app/node_modules/express/lib/router/index.js:312:13)
    at /app/node_modules/express/lib/router/index.js:280:7
    at Function.process_params (/app/node_modules/express/lib/router/index.js:330:12)
    at next (/app/node_modules/express/lib/router/index.js:271:10)
    at Function.handle (/app/node_modules/express/lib/router/index.js:176:3)

relevant change was made here: #2970

new RegExp(/\/\//, 'g') should just be /\/\//g
@wmertens
Copy link
Contributor

@jannispl could you do a quick PR?

@wmertens wmertens added the bug label Jul 25, 2016
@jannispl jannispl mentioned this issue Jul 25, 2016
Merged
@jannispl
Copy link
Author

@wmertens Done #3211

JedWatson added a commit that referenced this issue Jul 26, 2016
@JedWatson
Merge pull request #3211 from jannispl/patch-1
da3efe4 
Fix #3207
@JedWatson
Copy link
Member

Thanks, merged!

trongthanh added a commit to naustudio/keystone that referenced this issue Jan 16, 2018
@trongthanh
Merge commit 'c38b366c7b1bf01448c8ece7c36e1bb6eb195186' from mainstre…
5644ced 
…am/v0.3.x into release/two-language-main-vi

* commit 'c38b366c7b1bf01448c8ece7c36e1bb6eb195186': (63 commits)
  fixes Cloudinary select fields break other Relationship fields when using select: true (keystonejs#3848)
  Don't cache relationship options
  fix linter errors, I'm a little OCD
  implement fix to determine whether input param is string or object and act accordingly to set q  value
  Fix keystonejs#3207
  Removing hardcoded keystone references in the API
  0.3.22
  Update CHANGELOG
  Fix relationship inital display
  Allow local connections without forcing SSL
  Let's Encrypt support
  Fix to prevent the use of regex matching when applying an "exact" filter through the Keystone admin ui
  Convert req strings to number where needed
  keystonejs#3011 | Fix Date format option bug
  v0.3.21
  Updating changelog
  Only setting pretty local when it is undefined, ref keystonejs#2614
  v0.3.20
  Updating changelog
  keystonejs#3030 | fix Date unit tests
  ...

# Conflicts:
#	fields/types/boolean/BooleanField.js
#	index.js
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@wmertens @jannispl @JedWatson