)]}'
{"doc/requirements.txt":[{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"e96a68cfd5ccb34a5b72757c87d9cc6a792b752b","unresolved":true,"context_lines":[{"line_number":8,"context_line":"tenacity\u003e\u003d3.2.1 # Apache-2.0"},{"line_number":9,"context_line":"python-consul2\u003e\u003d0.1.5 # Apache-2.0"},{"line_number":10,"context_line":"Flask!\u003d0.11,\u003e\u003d1.0.2  # BSD"},{"line_number":11,"context_line":"Flask-RESTful\u003e\u003d0.3.5  # BSD"}],"source_content_type":"text/plain","patch_set":18,"id":"8a1c667b_3e991456","line":11,"updated":"2025-09-24 21:26:40.000000000","message":"Is that change mandatory?","commit_id":"2464e4e94afae41eab5419359139a6f1b4c281d5"},{"author":{"_account_id":36129,"name":"Xiang Wang","display_name":"Xiang(Roger) Wang","email":"rogerxiang.wang@gmail.com","username":"xiang-roger-wang"},"change_message_id":"ef32569df94a08b59baf7e0134b30f0af02c26a3","unresolved":false,"context_lines":[{"line_number":8,"context_line":"tenacity\u003e\u003d3.2.1 # Apache-2.0"},{"line_number":9,"context_line":"python-consul2\u003e\u003d0.1.5 # Apache-2.0"},{"line_number":10,"context_line":"Flask!\u003d0.11,\u003e\u003d1.0.2  # BSD"},{"line_number":11,"context_line":"Flask-RESTful\u003e\u003d0.3.5  # BSD"}],"source_content_type":"text/plain","patch_set":18,"id":"1c01388b_70876a1b","line":11,"in_reply_to":"16a67cb4_1a6806a7","updated":"2026-01-07 16:10:34.000000000","message":"thanks! i added it as an extra dependency too (in both pyproject.toml and requirments.txt)","commit_id":"2464e4e94afae41eab5419359139a6f1b4c281d5"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"19ec538bb39da349a7afe9a3a57222156f87f202","unresolved":true,"context_lines":[{"line_number":8,"context_line":"tenacity\u003e\u003d3.2.1 # Apache-2.0"},{"line_number":9,"context_line":"python-consul2\u003e\u003d0.1.5 # Apache-2.0"},{"line_number":10,"context_line":"Flask!\u003d0.11,\u003e\u003d1.0.2  # BSD"},{"line_number":11,"context_line":"Flask-RESTful\u003e\u003d0.3.5  # BSD"}],"source_content_type":"text/plain","patch_set":18,"id":"16a67cb4_1a6806a7","line":11,"in_reply_to":"8a1c667b_3e991456","updated":"2025-09-27 10:45:14.000000000","message":"I think yes because it is required by http driver code now, but this should be added as a extra dependency, too.","commit_id":"2464e4e94afae41eab5419359139a6f1b4c281d5"}],"oslo_messaging/_drivers/http_driver/flask/app/api.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"24398ce886f2c2a6e8183cd9b5774397d00d7305","unresolved":true,"context_lines":[{"line_number":59,"context_line":"        \"\"\""},{"line_number":60,"context_line":"        result \u003d None"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"        if request.is_json:"},{"line_number":63,"context_line":"            msg_id \u003d request.headers.get(\u0027x_msg_id\u0027, \u0027None\u0027)"},{"line_number":64,"context_line":"            try:"},{"line_number":65,"context_line":"                req_body \u003d request.get_json()"}],"source_content_type":"text/x-python","patch_set":18,"id":"646d7603_95b82726","line":62,"range":{"start_line":62,"start_character":8,"end_line":62,"end_character":27},"updated":"2025-09-27 10:59:49.000000000","message":"Do we expect any valid POST API call without json request body ? Then we should not ignore it but return an error.","commit_id":"2464e4e94afae41eab5419359139a6f1b4c281d5"},{"author":{"_account_id":36129,"name":"Xiang Wang","display_name":"Xiang(Roger) Wang","email":"rogerxiang.wang@gmail.com","username":"xiang-roger-wang"},"change_message_id":"ef32569df94a08b59baf7e0134b30f0af02c26a3","unresolved":false,"context_lines":[{"line_number":59,"context_line":"        \"\"\""},{"line_number":60,"context_line":"        result \u003d None"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"        if request.is_json:"},{"line_number":63,"context_line":"            msg_id \u003d request.headers.get(\u0027x_msg_id\u0027, \u0027None\u0027)"},{"line_number":64,"context_line":"            try:"},{"line_number":65,"context_line":"                req_body \u003d request.get_json()"}],"source_content_type":"text/x-python","patch_set":18,"id":"421d79f9_16d42f80","line":62,"range":{"start_line":62,"start_character":8,"end_line":62,"end_character":27},"in_reply_to":"646d7603_95b82726","updated":"2026-01-07 16:10:34.000000000","message":"That’s true. I removed the “if statement” so that an error can be raised and caught when “request.get_json()” is called.","commit_id":"2464e4e94afae41eab5419359139a6f1b4c281d5"}],"oslo_messaging/_drivers/http_driver/flask/openapi.yaml":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"d41b4240448df9e2c41bdc31972e2f0442cdd33c","unresolved":true,"context_lines":[{"line_number":1,"context_line":"openapi: 3.0.0"},{"line_number":2,"context_line":"info:"},{"line_number":3,"context_line":"  description: HTTP Driver RPC API"},{"line_number":4,"context_line":"  title: HTTP_RPC"}],"source_content_type":"text/x-yaml","patch_set":18,"id":"01703e28_08777fd6","line":1,"updated":"2025-09-27 10:43:01.000000000","message":"Do we still need this file within the code ? AFAIK you dropped connextion and introduced the static app instead.","commit_id":"2464e4e94afae41eab5419359139a6f1b4c281d5"},{"author":{"_account_id":36129,"name":"Xiang Wang","display_name":"Xiang(Roger) Wang","email":"rogerxiang.wang@gmail.com","username":"xiang-roger-wang"},"change_message_id":"ef32569df94a08b59baf7e0134b30f0af02c26a3","unresolved":false,"context_lines":[{"line_number":1,"context_line":"openapi: 3.0.0"},{"line_number":2,"context_line":"info:"},{"line_number":3,"context_line":"  description: HTTP Driver RPC API"},{"line_number":4,"context_line":"  title: HTTP_RPC"}],"source_content_type":"text/x-yaml","patch_set":18,"id":"1b8b7ce2_7aa10514","line":1,"in_reply_to":"01703e28_08777fd6","updated":"2026-01-07 16:10:34.000000000","message":"You\u0027re right. this is no longer needed. just removed it","commit_id":"2464e4e94afae41eab5419359139a6f1b4c281d5"}],"test-requirements.txt":[{"author":{"_account_id":36129,"name":"Xiang Wang","display_name":"Xiang(Roger) Wang","email":"rogerxiang.wang@gmail.com","username":"xiang-roger-wang"},"change_message_id":"8b2e2904e85aa07ca8e2e931b154d07442c26c72","unresolved":true,"context_lines":[{"line_number":11,"context_line":"# for test_impl_http"},{"line_number":12,"context_line":"python-consul2\u003e\u003d0.1.5 # Apache-2.0"},{"line_number":13,"context_line":"connexion[flask,swagger_ui]\u003e\u003d2.6.0,\u003c3.0.0 # Apache-2.0"},{"line_number":14,"context_line":"jsonschema\u003c\u003d4.19.0 # MIT"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"coverage\u003e\u003d4.0 # Apache-2.0"},{"line_number":17,"context_line":""}],"source_content_type":"text/plain","patch_set":1,"id":"147936bf_30493d9d","line":14,"updated":"2024-12-27 11:09:01.000000000","message":"This conflicts with the current upper constraints of `jsonschema\u003d\u003d\u003d4.19.2`.\nSince the current code needs to be under 4.19.0, a change in the upper constraints may be required.","commit_id":"9283fc3f1b57f77720cbb74dfcf47d3ce817365a"},{"author":{"_account_id":36129,"name":"Xiang Wang","display_name":"Xiang(Roger) Wang","email":"rogerxiang.wang@gmail.com","username":"xiang-roger-wang"},"change_message_id":"9945a78165c386663a4e9daa8d1f4e964f6820f8","unresolved":true,"context_lines":[{"line_number":11,"context_line":"# for test_impl_http"},{"line_number":12,"context_line":"python-consul2\u003e\u003d0.1.5 # Apache-2.0"},{"line_number":13,"context_line":"connexion[flask,swagger_ui]\u003e\u003d2.6.0,\u003c3.0.0 # Apache-2.0"},{"line_number":14,"context_line":"jsonschema\u003c\u003d4.19.0 # MIT"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"coverage\u003e\u003d4.0 # Apache-2.0"},{"line_number":17,"context_line":""}],"source_content_type":"text/plain","patch_set":1,"id":"7221903f_afb259f4","line":14,"in_reply_to":"147936bf_30493d9d","updated":"2025-01-07 08:00:22.000000000","message":"This current code version doesn\u0027t work with the current upper-constraints.txt\n\nit has only been tested with the following versions:\n\n- jsonschema \u003c\u003d 4.19.0\n- flask \u003c 2.2.3\n- Werkzeug \u003d\u003d 2.2.3\n\nwe need to resolve the version conflicts.","commit_id":"9283fc3f1b57f77720cbb74dfcf47d3ce817365a"},{"author":{"_account_id":36129,"name":"Xiang Wang","display_name":"Xiang(Roger) Wang","email":"rogerxiang.wang@gmail.com","username":"xiang-roger-wang"},"change_message_id":"8d220115110e2846370aa7631d27f6c88700e1e2","unresolved":false,"context_lines":[{"line_number":11,"context_line":"# for test_impl_http"},{"line_number":12,"context_line":"python-consul2\u003e\u003d0.1.5 # Apache-2.0"},{"line_number":13,"context_line":"connexion[flask,swagger_ui]\u003e\u003d2.6.0,\u003c3.0.0 # Apache-2.0"},{"line_number":14,"context_line":"jsonschema\u003c\u003d4.19.0 # MIT"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"coverage\u003e\u003d4.0 # Apache-2.0"},{"line_number":17,"context_line":""}],"source_content_type":"text/plain","patch_set":1,"id":"b936287f_d6bbfcbf","line":14,"in_reply_to":"7221903f_afb259f4","updated":"2025-03-24 06:05:52.000000000","message":"These dependency conflicts have been completely resolved by re-implementing http-server and client purely using flask instead of auto-generated code based on connexion[flask]","commit_id":"9283fc3f1b57f77720cbb74dfcf47d3ce817365a"}],"tox.ini":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"2b8c0aaa9584423e9c1499ca2ec23bfb88b6ef5d","unresolved":true,"context_lines":[{"line_number":69,"context_line":"commands \u003d"},{"line_number":70,"context_line":"  {toxinidir}/tools/setup-scenario-env.sh stestr run --slowest {posargs:oslo_messaging.tests.functional}"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"# The following functional tests for http-driver require consul to be installed."},{"line_number":73,"context_line":"[testenv:py310-func-http]"},{"line_number":74,"context_line":"setenv \u003d"},{"line_number":75,"context_line":"    ENVNAME\u003d{envname}"}],"source_content_type":"text/x-properties","patch_set":18,"id":"57c6ed62_81e18a9a","line":72,"updated":"2025-09-27 10:46:26.000000000","message":"Please also update zuul config to add a new test with this target actually used.","commit_id":"2464e4e94afae41eab5419359139a6f1b4c281d5"},{"author":{"_account_id":36129,"name":"Xiang Wang","display_name":"Xiang(Roger) Wang","email":"rogerxiang.wang@gmail.com","username":"xiang-roger-wang"},"change_message_id":"ef32569df94a08b59baf7e0134b30f0af02c26a3","unresolved":false,"context_lines":[{"line_number":69,"context_line":"commands \u003d"},{"line_number":70,"context_line":"  {toxinidir}/tools/setup-scenario-env.sh stestr run --slowest {posargs:oslo_messaging.tests.functional}"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"# The following functional tests for http-driver require consul to be installed."},{"line_number":73,"context_line":"[testenv:py310-func-http]"},{"line_number":74,"context_line":"setenv \u003d"},{"line_number":75,"context_line":"    ENVNAME\u003d{envname}"}],"source_content_type":"text/x-properties","patch_set":18,"id":"66be9e51_e5a2480d","line":72,"in_reply_to":"57c6ed62_81e18a9a","updated":"2026-01-07 16:10:34.000000000","message":"Done","commit_id":"2464e4e94afae41eab5419359139a6f1b4c281d5"}]}