ci_lab Integration candidate: Caelum-rc4+dev64 #155

dzbaker · 2023-11-13T19:59:30Z

Checklist (Please check before submitting)

I reviewed the Contributing Guide.
I signed and emailed the appropriate Contributor License Agreement to GSFC-SoftwareRelease@mail.nasa.gov and copied cfs-program@lists.nasa.gov.

Describe the contribution

PR Fix #149, add decode hooks #152. Addresses Issue Add hooks for decoding/translating of network buffer #149
PR Fix #52, Add version information to NOOP event #153. Addresses Issue Report version when responding to NoOp Comand to match Sample App template #52

Testing performed
cFS Bundle Checks
ci_lab Checks

Expected behavior changes
See PRs

System(s) tested on
Ubuntu 22.04

Additional context
Add any other context about the contribution here.

Third party code
If included, identify any third party code and provide text file of license

Contributor Info - All information REQUIRED for consideration of pull request
@jphickey
@thnkslprpt

Add an intermediate step during ingest to allow the buffer to be decoded prior to sending to SB. Initial implementation is just a pass-through, thus matching existing behavior.

Fix #149, add decode hooks

…n-to-noop-events Fix #52, Add version information to NOOP event

fsw/src/ci_lab_passthru_decode.c

+ * otherwise, the source buffer is used directly as the output buffer.
+ * ---------------------------------------
+ */
+CFE_Status_t CI_LAB_DecodeInputMessage(void *SourceBuffer, size_t SourceSize, CFE_SB_Buffer_t **DestBufferOut)


fsw/src/ci_lab_passthru_decode.c

+ * SB and put it directly in there.  This reduces copying.
+ * ---------------------------------------
+ */
+CFE_Status_t CI_LAB_GetInputBuffer(void **BufferOut, size_t *SizeOut)


jphickey and others added 5 commits November 9, 2023 15:54

Fix #149, add decode hooks

a421335

Add an intermediate step during ingest to allow the buffer to be decoded prior to sending to SB. Initial implementation is just a pass-through, thus matching existing behavior.

Fix #52, Add version information to NOOP event

ea9219f

Merge pull request #152 from jphickey:fix-149-decode

a7d1d05

Fix #149, add decode hooks

Merge pull request #153 from thnkslprpt:fix-52-add-version-informatio…

dd837b1

…n-to-noop-events Fix #52, Add version information to NOOP event

Updating documentation and version numbers for v2.5.0-rc4+dev69

baed83d

github-advanced-security bot found potential problems Nov 13, 2023

View reviewed changes

dzbaker merged commit 6e55bb4 into main Nov 14, 2023
18 of 19 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

ci_lab Integration candidate: Caelum-rc4+dev64 #155

ci_lab Integration candidate: Caelum-rc4+dev64 #155

dzbaker commented Nov 13, 2023 •

edited

Loading

ci_lab Integration candidate: Caelum-rc4+dev64 #155

ci_lab Integration candidate: Caelum-rc4+dev64 #155

Conversation

dzbaker commented Nov 13, 2023 • edited Loading

dzbaker commented Nov 13, 2023 •

edited

Loading