search

google / cap-library

Common Alerting Protocol Library
Apache License 2.0
80 stars 30 forks source link

Add URI and string fields validation #55

Closed sschiavoni closed 9 years ago

shakusa commented 9 years ago

Would you mind throwing in the cap-validator -> cap-validator-hrd change to the validator's appengine-web.xml?

sschiavoni commented 9 years ago

@shakusa I already submitted the change for the App identifier at HEAD (https://github.com/google/cap-library/blob/master/validator/war/WEB-INF/appengine-web.xml). This change needs to be merged. Is this what you were referring to?

sschiavoni commented 9 years ago

@shakusa Concerning the weather.atom: yes, I'm going to rollback my rollback and make sure the file is identical to the one we have internally. I didn't realize that it was recently changed, and I didn't understand what was happening.

sschiavoni commented 9 years ago

@shakusa PTAL

shakusa commented 9 years ago

LGTM

Both comments are resolved. Thanks!