fhir-data-pipes
fhir-data-pipes copied to clipboard
Remove path variable when retrieving the error log file.
Description of what I changed
Currently to retrieve the error log file associated with the latest pipeline run, we pass the log file path in the request as a path parameter, this is prone to vulnerability attacks as there are no checks to restrict the paths only to the data-warehouse directory.
This PR contains changes to remove the path parameter altogether and only retrieves the error log associated with the latest pipeline run if exists.
E2E test
Tested the changed api and it works as expected. Also, the path parameter to accept file path is no more valid.
TESTED:
relied on e2e tests.
Checklist: I completed these to help reviewers :)
-
[x] I have read and will follow the review process.
-
[x] I am familiar with Google Style Guides for the language I have coded in.
No? Please take some time and review Java and Python style guides.
-
[x] My IDE is configured to follow the Google code styles.
No? Unsure? -> configure your IDE.
-
[ ] I have added tests to cover my changes. (If you refactored existing code that was well tested you do not have to add tests)
-
[x] I ran
mvn clean packageright before creating this pull request and added all formatting changes to my commit. -
[x] All new and existing tests passed.
-
[x] My pull request is based on the latest changes of the master branch.
No? Unsure? -> execute command
git pull --rebase upstream master
Codecov Report
Attention: Patch coverage is 0% with 9 lines in your changes missing coverage. Please review.
Project coverage is 52.50%. Comparing base (
c92ff49) to head (4e73b17).
| Files | Patch % | Lines |
|---|---|---|
| .../java/com/google/fhir/analytics/ApiController.java | 0.00% | 8 Missing :warning: |
| ...ava/com/google/fhir/analytics/PipelineManager.java | 0.00% | 1 Missing :warning: |
Additional details and impacted files
@@ Coverage Diff @@
## master #1139 +/- ##
============================================
- Coverage 52.59% 52.50% -0.09%
Complexity 706 706
============================================
Files 95 95
Lines 5751 5757 +6
Branches 765 767 +2
============================================
- Hits 3025 3023 -2
- Misses 2434 2443 +9
+ Partials 292 291 -1
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.