Remove Integer from PaseVerifierParams constructor #20586
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The spec was updated to force PasscodeId to be always zero for v1.0, and this was updated in project-chip#15924. This change removes it from the corresponding JNI constructor which was missed in the original fix.
|
|
…#20546) With all three SDKs installed, the imx docker image becomes 33.5GB and exceeds the size limit of github CI. remove two SDKs and keep the the SDK of 5.15 kernel for i.MX 8mm devices. The generated docker image is now 17GB. Change-Id: I20ac1d1052cc4a9707ea2d58757e3bba47dbcd69 Signed-off-by: faqiang.zhu <[email protected]>
…p#20304)" (project-chip#20563) This reverts commit 757682e.
* upgrade zap * run codegen
…chip#20581) This got lost during the rename.
* add CI for Bouffalolab BL602 * add newline at end of file * use build_examples.py to build lighting-app * use chip-build docker image for bl602 * fix build error
… in door-lock-server.cpp (project-chip#20538) * Fix Chef DoorLock lock/unlock functions didn't overide weak functions in door-lock-server.cpp * Fix restyle issue in src/app/chip_data_model.gni * Restyled by whitespace * Restyled by clang-format * Restyled by gn Co-authored-by: Restyled.io <[email protected]>
…FAILURE) (project-chip#20553) * Mark constraint error as the error to return on overly long values * Zap regen * Fix return code * Update TC-ULABEL-2.3 for using constraint_error * Zap regen
…oject-chip#20570) * Test added march 8 (project-chip#15957) * Added new manual scripts * Added Auto generated File * [OTA] Fix OTARequestorDriverImpl inclusion (project-chip#15981) * Regen to fix CI failures (project-chip#15990) * [ota] Store Default OTA Providers in flash (project-chip#15970) * [ota] Store Default OTA Providers in flash Store Default OTA Providers in flash each time the attribute is modified and load it back on the application startup. * Restyled by clang-format * Fix build and reduce flash usage Co-authored-by: Restyled.io <[email protected]> * Set MRP Active Retry interval in the Linux ota-provider-app config * Remove merge artifacts * Restyled by whitespace Co-authored-by: kowsisoundhar12 <[email protected]> Co-authored-by: Carol Yang <[email protected]> Co-authored-by: Boris Zbarsky <[email protected]> Co-authored-by: Damian Królik <[email protected]> Co-authored-by: Restyled.io <[email protected]>
) I forgot to update the factory reset routine after adding escaping of forbidden characters in KVS keys and the routine is escaping such keys twice which results in a failure. Signed-off-by: Damian Krolik <[email protected]>
- Don't use -isystem for project configs as this suppresses warnings - Use relative paths for options as is a best practice
…`FileNotFoundError: [Errno 2] No such file or directory: 'mbed-tools'` (project-chip#20616)
They are unused. Fixes project-chip#20221
Introduce a new variable p6_sdk_root for the actual root of the SDK repository and use this to specify SDK paths. This avoids forcing locating the SDK submodule at a particular location in the Matter SDK directory, as projects may want to use a different layout.
* Fixed Door lock issues * Added Auto generated files * Restyled by whitespace * Restyled by clang-format * Restyled by prettier-yaml Co-authored-by: Restyled.io <[email protected]>
…roject-chip#20609) It is still not agreed if certification declaration that is part of a firmware will be allowed to change after certification. It might happen that the CD will need to be changed some other way than modifying the firmware. * Added config option allowing to enable certification declaration storage. * Added handler that is able to read CD from OTA DFU image and save it in the Zephyr settings. * Added nRFConnect define to set the Certification Declaration value and not use the hardcoded one.
…roject-chip#20637) The Darwin framework was using the current timezone, not UTC, when determining the Matter epoch time corresponding to a given offset from now. This caused the epoch times it computed to be off by the offset from UTC. In timezones ahead of UTC, this could easily lead to certificates with mNotBeforeTime set to a value larger than the current UTC time, which would then cause those certificates to be considered not-yet-valid. Fixes project-chip#20302
) Change-Id: Ide950881e49a29351a43354738845a391e0148d4 Signed-off-by: faqiang.zhu <[email protected]>
) * add support for coverage enabled apps in build_examples.py * Switch to gcc builds * Coverage generation support * Fix typo * Update ordering for builds * Restyle * Update scripts/build/builders/gn.py Co-authored-by: Terence Hampson <[email protected]> Co-authored-by: Terence Hampson <[email protected]>
* Update OTA image tool - allow removing headers - allow re-writing headers * Read/write in chunks
* Modified scripts TC-I-1.1 TC-LVL-3.1 TC-LVL-4.1 TC-LVL-6.1 TC-OO-1.1 * Auto generated files
boring-cyborg
bot
added
app
config
examples
github
integrations
scripts
vscode
workflows
labels
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Problem
The spec was updated to force PasscodeId to be always zero for v1.0, and this was updated in #15924. However, a corresponding JNI constructor was missed in the original fix.
Change overview
Testing