Skip to content

chore: add data_access support for encrypted endpoints#1993

Open
frossq wants to merge 1 commit into
astarte-platform:masterfrom
frossq:enc-endp-data-access
Open

chore: add data_access support for encrypted endpoints#1993
frossq wants to merge 1 commit into
astarte-platform:masterfrom
frossq:enc-endp-data-access

Conversation

@frossq
Copy link
Copy Markdown
Collaborator

@frossq frossq commented May 27, 2026

Allow queries to recognize encrypted option in mappings and encrypted data values

What this PR does / why we need it:

Additional documentation e.g. usage docs, diagrams, reviewer notes, etc.:


Thanks for sending a pull request! If this is your first time, here are some tips for you:
  1. You can take a look at our developer guide for an introduction on Astarte development!
  2. Make sure to read CONTRIBUTING.md and CODE_OF_CONDUCT.md
  3. If the PR is unfinished or you're actively working on it, mark it as draft

When fixing existing issues, use github's syntax to link your pull request to it

fixes #<issue number>

We also have a syntax to signal dependencies to other open pull requests

depends on #<pr number>
depends on https://github.com/...

In case of stacked PRs, you may add the PR number in the last commit's title instead:

gitGraph
    commit id: "Current master"
    branch feat1
    checkout feat1
    commit id: "feat: add something"
    commit id: "feat: add something else (#100)"
    branch feat2
    checkout feat2
    commit id: "refactor: do something"
    commit id: "fix: solve issue"
    commit id: "feat: add a feature (#101)"
    branch feat3
    checkout feat3
    commit id: "feat: feat without pr number"
Loading

@codecov
Copy link
Copy Markdown

codecov Bot commented May 27, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.69%. Comparing base (7db95a9) to head (5c26346).
⚠️ Report is 3825 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff             @@
##           master    #1993       +/-   ##
===========================================
+ Coverage   67.67%   86.69%   +19.02%     
===========================================
  Files         264      678      +414     
  Lines        6456    11973     +5517     
===========================================
+ Hits         4369    10380     +6011     
+ Misses       2087     1593      -494     
Files with missing lines Coverage Δ
...ne_api/lib/astarte_appengine_api/device/queries.ex 95.20% <ø> (+13.89%) ⬆️
...te_data_access/lib/astarte_data_access/mappings.ex 100.00% <ø> (ø)
..._access/lib/astarte_data_access/realms/endpoint.ex 100.00% <ø> (ø)
...starte_data_access/realms/individual_datastream.ex 0.00% <ø> (ø)
.../astarte_data_access/realms/individual_property.ex 66.66% <ø> (ø)

... and 558 files with indirect coverage changes

Flag Coverage Δ
astarte_adapters 100.00% <ø> (?)
astarte_appengine_api 86.31% <ø> (+3.82%) ⬆️
astarte_data_access 84.97% <ø> (?)
astarte_data_access_generators 100.00% <ø> (?)
astarte_data_updater_plant 87.15% <ø> (+23.15%) ⬆️
astarte_events 80.27% <ø> (?)
astarte_fdo 81.53% <ø> (?)
astarte_fdo_core 77.99% <ø> (?)
astarte_generators 97.74% <ø> (?)
astarte_housekeeping 86.39% <ø> (+38.55%) ⬆️
astarte_realm_management 88.64% <ø> (+22.33%) ⬆️
astarte_rpc 88.23% <ø> (?)
astarte_secrets 87.73% <ø> (?)
astarte_test_suite 100.00% <ø> (?)
astarte_trigger_engine 82.67% <ø> (+38.69%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@frossq frossq force-pushed the enc-endp-data-access branch from c4514e8 to 4f84dc3 Compare May 27, 2026 12:34
@frossq frossq force-pushed the enc-endp-data-access branch 3 times, most recently from 2371897 to 4502afd Compare May 27, 2026 13:56
Allow queries to recognize encrypted option in mappings
Add new db fields in test setup for all services

Signed-off-by: frossq <francesco.asquini@secomind.com>
@frossq frossq force-pushed the enc-endp-data-access branch from 4502afd to 5c26346 Compare May 27, 2026 14:37
@frossq frossq marked this pull request as ready for review May 27, 2026 14:48
@frossq frossq requested review from eddbbt, nedimtokic and noaccOS May 27, 2026 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants