Skip to content

Bump backend and cli submodules to include fix for #1062 - #1066

Merged
rquidute merged 2 commits into
v2.15.1-beta1+summer2026from
fix/1062-bump-backend-cli-submodules
Jul 22, 2026
Merged

rquidute merged 2 commits into
v2.15.1-beta1+summer2026from
fix/1062-bump-backend-cli-submodules

Conversation

@rquidute

Copy link
Copy Markdown
Contributor

Summary

Bumps the backend and cli submodules to include the merged fixes for #1062 (THCLI reports a WebSocket error after successfully uploading a manual test log, e.g. TC-CADMIN-1.17).

Submodule changes

  • backend f49b828..cb44885 — certification-tool-backend#343: batches uploaded log lines into chunks of 500 before logging (instead of one logger.info() call per line, which stalled the event loop long enough to trip the WebSocket's keepalive ping/pong), and raises the production ws_ping_timeout to 60s to match the existing dev-only setting.
  • cli 79cb747..0f00720 — certification-tool-cli#105: stops misreporting a successful upload as ❌ Unexpected error uploading file when sending the confirmation fails after the file was already uploaded, and strips stray whitespace/CR from the uploaded file path prompt input (a separate bug found while verifying this fix, where a trailing \r from some terminals caused a valid, existing file path to be rejected).

Note: the cli bump also pulls in an unrelated, already-merged commit (79cb747..0f00720 includes PR #103, a Push AV Stream Verification fix) since it was merged into v2.15.1-cli-develop between when this work started and now.

Testing

Both submodule PRs were reviewed, had review feedback addressed, and merged individually with unit test coverage. This PR is a pointer-only change (no other file modifications).

Pulls in the merged fixes for the WebSocket disconnect that occurred
during large manual log file uploads (e.g. TC-CADMIN-1.17):

- backend f49b828..cb44885: batches uploaded log lines into chunks
  before logging (instead of one logger.info() call per line, which
  stalled the event loop long enough to trip the WebSocket's
  keepalive), and raises the production ws_ping_timeout to 60s
  (certification-tool-backend#343).
- cli 79cb747..0f00720: stops misreporting a successful upload as
  "Unexpected error uploading file" when the confirmation send fails
  after the file was already uploaded, and strips stray whitespace/CR
  from the uploaded file path prompt input (certification-tool-cli#105).

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the subproject commits for both the backend and cli directories to newer versions. There are no review comments, and I have no feedback to provide.

@mergify

mergify Bot commented Jul 22, 2026

Copy link
Copy Markdown

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@rquidute rquidute self-assigned this Jul 22, 2026
@rquidute
rquidute requested a review from antonio-amjr July 22, 2026 19:40
@rquidute
rquidute merged commit 962335a into v2.15.1-beta1+summer2026 Jul 22, 2026
4 checks passed
@rquidute
rquidute deleted the fix/1062-bump-backend-cli-submodules branch July 22, 2026 19:46
@rquidute
rquidute restored the fix/1062-bump-backend-cli-submodules branch July 24, 2026 18:18
oxesoft pushed a commit that referenced this pull request Aug 20, 2026
* Bump backend and cli submodules to include fix for #1062

Pulls in the merged fixes for the WebSocket disconnect that occurred
during large manual log file uploads (e.g. TC-CADMIN-1.17):

- backend f49b828..cb44885: batches uploaded log lines into chunks
  before logging (instead of one logger.info() call per line, which
  stalled the event loop long enough to trip the WebSocket's
  keepalive), and raises the production ws_ping_timeout to 60s
  (certification-tool-backend#343).
- cli 79cb747..0f00720: stops misreporting a successful upload as
  "Unexpected error uploading file" when the confirmation send fails
  after the file was already uploaded, and strips stray whitespace/CR
  from the uploaded file path prompt input (certification-tool-cli#105).

* Update docker-compose.yml backend image tag to cb44885 to match submodule
# Conflicts:
#	backend
#	cli
#	docker-compose.yml
@oxesoft

oxesoft commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Cherry-picked to v2.16-beta2+summer2026

@rquidute
rquidute deleted the fix/1062-bump-backend-cli-submodules branch September 3, 2026 17:31
rquidute added a commit that referenced this pull request Sep 10, 2026
* [Improve UX] Suppress raw Docker errors when pulling backend/frontend images from GHCR (#1013)

* [Improve UX] Suppress raw Docker errors when pulling backend/frontend images from GHCR

* Code review - Google Code assist

* User guide update 1.6 (#1015)

* Update links and version information in user guide

* Update Causeway URL for 1.6

* Feature/1016 Create pics to xml script (#1019)

* Created pics_to_xml.sh tool script

* Removed unwanted files

* Cpde review - Google Code assist

* Update frontend image and submodule to 10d1822 (#1034)

* Release v2.15.1-beta1+summer2026 (#1045)

* Release v2.15.1-beta1+summer2026

- Pin backend submodule to bfb0a49 (v2.15.1-develop tip)
- Update docker-compose backend image tag to bfb0a49
- Update Matter TH User Guide: th-version to v2.15.1-beta1+summer2026,
  add Matter 1.6.1 row with SDK commit df8bd03

* Updated UG PDF file

* Updated CLI

* Fix #1044: backend image build (npm/Node incompatibility) + CommissionDeviceTest import (#1048)

- Pin backend submodule to 3ed2c13 (certification-tool-backend#340):
  removes the incompatible nodejs/npm/cspell install from the Dockerfile,
  and fixes the CommissionDeviceTest import path for the current SDK pin
  (df8bd030 / v1.6.1-mve-branch moved it back to
  matter.testing.CommissioningPreTest).
- Update docker-compose.yml backend image tag to 3ed2c13 to match.

* Enhance user guide with hostname usage in the Quick start guide (#1052)

* Clarify hostname usage in Quick-Start Guide

Added clarification on hostname usage in Quick-Start Guide section and updated related documentation.

* Update hostname clarification in Quick-Start Guide

* updated user guide PDF

* Fix grammar in hostname configuration instructions

* updated user guide PDF with grammatical fix

* updated user guide PDF with grammatical fix

* Bump cli submodule to include fix for #1051 (#1058)

Bumps the cli submodule from 34de9c3 to 79cb747 (project-chip/certification-tool-cli#103),
which fixes the Push AV Stream Verification page reading stale field
names (files/valid_files/invalid_files) instead of the current
valid_uploads/error_uploads shape returned by the Push AV Server.

Fixes #1051

* Document running side-loaded Python scripts inside container (#1059)

* Fix #1054: document running side-loaded Python scripts inside container

Add a new User Guide subsection under "Test-Harness Side Load Feature
with Custom Test Cases" explaining how to manually run a side-loaded
Python script directly inside the container when it imports shared
SDK helper modules (e.g. TC_GC_common.py) that only live in the
sibling sdk/ scripts folder.

The Test-Harness's normal run-tests flow (CLI/UI) already makes these
imports resolve correctly on its own. Manual invocation inside the
container needs PYTHONPATH set to the sdk scripts folder, otherwise
it fails with ModuleNotFoundError.

* Added pdf file

* Bump backend and cli submodules to include fix for #1062 (#1066)

* Bump backend and cli submodules to include fix for #1062

Pulls in the merged fixes for the WebSocket disconnect that occurred
during large manual log file uploads (e.g. TC-CADMIN-1.17):

- backend f49b828..cb44885: batches uploaded log lines into chunks
  before logging (instead of one logger.info() call per line, which
  stalled the event loop long enough to trip the WebSocket's
  keepalive), and raises the production ws_ping_timeout to 60s
  (certification-tool-backend#343).
- cli 79cb747..0f00720: stops misreporting a successful upload as
  "Unexpected error uploading file" when the confirmation send fails
  after the file was already uploaded, and strips stray whitespace/CR
  from the uploaded file path prompt input (certification-tool-cli#105).

* Update docker-compose.yml backend image tag to cb44885 to match submodule

* Release v2.15.1+summer2026 (#1081)

* Release v2.15.1+summer2026

* PDF file

* Release v2.15.1+summer2026 (#1090)

Pins backend to v2.15.1-develop tip (4cb5753), which includes the
fix for #358 (Mandatory SDK Python Tests duplicated into the Old
Script Format collection, causing double execution via the TH CLI).

Frontend and cli are unchanged (already at their respective release
tips).

* Update Causeway link for Matter 1.6.1 (#1096)

* Update Causeway link for Matter 1.6.1

Updated the Causeway link for Matter 1.6.1 in the Test-Harness Links section.

* updated user guide PDF

* Release v2.15.1+summer2026 (frontend bump) (#1098)

---------

Co-authored-by: Romulo Quidute Filho <116586593+rquidute@users.noreply.github.com>
Co-authored-by: Kishok G <133193761+KishokG@users.noreply.github.com>
Co-authored-by: Daniel Moura <oxesoft@gmail.com>
Co-authored-by: Pradeep-grl <pradeep.g@graniteriverlabs.in>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants