[Hawkular-commits] [hawkular/hawkular-alerts] 1510aa: HWKALERTS-239 Query parameter validation

Lucas Ponce ponce.ballesteros at gmail.com
Thu May 4 08:15:05 EDT 2017


  Branch: refs/heads/master
  Home:   https://github.com/hawkular/hawkular-alerts
  Commit: 1510aa8bcaa7328885326a5011a46c5c0485026f
      https://github.com/hawkular/hawkular-alerts/commit/1510aa8bcaa7328885326a5011a46c5c0485026f
  Author: Jay Shaughnessy <jshaughn at redhat.com>
  Date:   2017-05-03 (Wed, 03 May 2017)

  Changed paths:
    M hawkular-alerts-rest-tests/src/test/groovy/org/hawkular/alerts/rest/AbstractITestBase.groovy
    M hawkular-alerts-rest-tests/src/test/groovy/org/hawkular/alerts/rest/AlertsITest.groovy
    M hawkular-alerts-rest-tests/src/test/groovy/org/hawkular/alerts/rest/CORSITest.groovy
    M hawkular-alerts-rest-tests/src/test/groovy/org/hawkular/alerts/rest/EventsITest.groovy
    M hawkular-alerts-rest/hawkular-alerts-rest-api/src/main/java/org/hawkular/alerts/rest/ActionsHandler.java
    M hawkular-alerts-rest/hawkular-alerts-rest-api/src/main/java/org/hawkular/alerts/rest/AlertsHandler.java
    M hawkular-alerts-rest/hawkular-alerts-rest-api/src/main/java/org/hawkular/alerts/rest/CrossTenantHandler.java
    M hawkular-alerts-rest/hawkular-alerts-rest-api/src/main/java/org/hawkular/alerts/rest/EventsHandler.java
    A hawkular-alerts-rest/hawkular-alerts-rest-api/src/main/java/org/hawkular/alerts/rest/QueryParamValidation.java
    M hawkular-alerts-rest/hawkular-alerts-rest-api/src/main/java/org/hawkular/alerts/rest/RequestUtil.java
    M hawkular-alerts-rest/hawkular-alerts-rest-api/src/main/java/org/hawkular/alerts/rest/ResponseUtil.java
    M hawkular-alerts-rest/hawkular-alerts-rest-api/src/main/java/org/hawkular/alerts/rest/TriggersHandler.java

  Log Message:
  -----------
  HWKALERTS-239 Query parameter validation
- Apply query param validation to all criteria-based services
- add new QueryParamValidation annotation
- add new ResponseUtil.populateQueryParamsMap util for gathering
    query param names for annotated services.
- add new ResponseUtil.checkForUnknownQueryParams for validating requests


  Commit: d9ec4ae43011e09ed599fb87072ceebfae9c18d1
      https://github.com/hawkular/hawkular-alerts/commit/d9ec4ae43011e09ed599fb87072ceebfae9c18d1
  Author: Lucas Ponce <ponce.ballesteros at gmail.com>
  Date:   2017-05-04 (Thu, 04 May 2017)

  Changed paths:
    M hawkular-alerts-rest-tests/src/test/groovy/org/hawkular/alerts/rest/AbstractITestBase.groovy
    M hawkular-alerts-rest-tests/src/test/groovy/org/hawkular/alerts/rest/AlertsITest.groovy
    M hawkular-alerts-rest-tests/src/test/groovy/org/hawkular/alerts/rest/CORSITest.groovy
    M hawkular-alerts-rest-tests/src/test/groovy/org/hawkular/alerts/rest/EventsITest.groovy
    M hawkular-alerts-rest/hawkular-alerts-rest-api/src/main/java/org/hawkular/alerts/rest/ActionsHandler.java
    M hawkular-alerts-rest/hawkular-alerts-rest-api/src/main/java/org/hawkular/alerts/rest/AlertsHandler.java
    M hawkular-alerts-rest/hawkular-alerts-rest-api/src/main/java/org/hawkular/alerts/rest/CrossTenantHandler.java
    M hawkular-alerts-rest/hawkular-alerts-rest-api/src/main/java/org/hawkular/alerts/rest/EventsHandler.java
    A hawkular-alerts-rest/hawkular-alerts-rest-api/src/main/java/org/hawkular/alerts/rest/QueryParamValidation.java
    M hawkular-alerts-rest/hawkular-alerts-rest-api/src/main/java/org/hawkular/alerts/rest/RequestUtil.java
    M hawkular-alerts-rest/hawkular-alerts-rest-api/src/main/java/org/hawkular/alerts/rest/ResponseUtil.java
    M hawkular-alerts-rest/hawkular-alerts-rest-api/src/main/java/org/hawkular/alerts/rest/TriggersHandler.java

  Log Message:
  -----------
  Merge pull request #307 from jshaughn/hwkalerts-239

HWKALERTS-239


Compare: https://github.com/hawkular/hawkular-alerts/compare/d2e73c68a10a...d9ec4ae43011


More information about the hawkular-commits mailing list