Skip to content

Commit

Permalink
temporary revert cornice.Service.pyramid_route for route_prefix to re…
Browse files Browse the repository at this point in the history
…solve multi-config issue in tests (relates to Cornices/cornice#584)
  • Loading branch information
fmigneault committed May 2, 2024
1 parent 8488f02 commit b1e7791
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 2 deletions.
8 changes: 7 additions & 1 deletion CHANGES.rst
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,12 @@ Changes
`Unreleased <https://github.com/crim-ca/weaver/tree/master>`_ (latest)
========================================================================

.. FIXME:
- Resolve `Configurator.add_cornice_service` vs `cornice-swagger` `route_prefix` resolution for listing OpenAPI
endpoints with the appropriate prefix (https://github.com/Cornices/cornice/pull/584#issuecomment-2089676743).
- Add test validating that `route_prefix` is applied for HTTP requests.
- Add test validating that `route_prefix` is applied in paths of the resulting OpenAPI JSON from CorniceSwagger.
Changes:
--------
- Add support of `HTML` responses for `OGC API - Processes` endpoints
Expand All @@ -22,7 +28,7 @@ Changes:
- Add missing documentation for ``weaver.wps_restapi_doc`` and ``weaver.wps_restapi_ref`` configuration settings.
- Modified the base path/URL resolution of the `OpenAPI` endpoint to be located at the application root instead of being
nested under ``weaver.wps_restapi_path`` or ``weaver.wps_restapi_url``, since the OpenAPI `JSON` and `HTML` responses
are employed for representing supported requests and responses both the `REST` and the `OWS` `WPS` interfaces.
are employed for representing supported requests and responses of both the `REST` and the `OWS` `WPS` interfaces.
- Update `Swagger-UI` version for latest rendering fixes of `OpenAPI` definitions.
- Add multiple missing `OGC API - Processes` conformance references.
- Modify default query parameter value ``links=true`` for ``/processes`` summary listing to conform with
Expand Down
4 changes: 3 additions & 1 deletion requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,9 @@ celery[mongodb]>=5.2.2,<6; sys_platform != "win32"
cffi
colander<2
# FIXME: patch fix of the cornice service prefixed route path resolution (https://github.com/Cornices/cornice/pull/584)
cornice @ git+https://github.com/fmigneault/cornice.git@fix-service-route-prefix
# FIXME: issue during tests (see https://github.com/Cornices/cornice/pull/584#issuecomment-2089676743)
### cornice @ git+https://github.com/fmigneault/cornice.git@fix-service-route-prefix
cornice
#cornice_swagger>=0.7.0
cornice-swagger @ git+https://github.com/fmigneault/cornice.ext.swagger.git@openapi-3
cryptography
Expand Down

0 comments on commit b1e7791

Please sign in to comment.