Skip to content
This repository was archived by the owner on Nov 22, 2023. It is now read-only.

Allow : in names #250

Merged
merged 1 commit into from
Aug 17, 2016
Merged

Allow : in names #250

merged 1 commit into from
Aug 17, 2016

Conversation

csstaub
Copy link
Contributor

@csstaub csstaub commented Aug 17, 2016

Allow : in names

@worldwise001
Copy link
Contributor

👍 :shipit:

@alokmenghrajani
Copy link
Contributor

alokmenghrajani commented Aug 17, 2016

Move this code to SecretController.java and validate server side? (as a result, you will fix #170)

@csstaub
Copy link
Contributor Author

csstaub commented Aug 17, 2016

@alokmenghrajani Yes, we should probably do that; but in the meantime I need this fix to get in because... reasons.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 72.151% when pulling 3890f23 on cs/allow-colon into f17cf9d on master.

@csstaub csstaub merged commit e077bb0 into master Aug 17, 2016
@csstaub csstaub deleted the cs/allow-colon branch August 17, 2016 23:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants