-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Sanitise input data #200
Merged
+1,609
−626
Merged
Sanitise input data #200
Changes from 1 commit
Commits
Show all changes
48 commits
Select commit
Hold shift + click to select a range
bed9892
Remove create_output_folder() function
alexdewar 50505e1
Use std::filesystem::path cf. std::string in a few places
alexdewar 2700324
Drop support for v1 config files
alexdewar 358c762
load_configuration(): Remove some indentation
alexdewar a2fd569
Make separate function for rebasing paths
alexdewar c66c0b8
Put new functions for loading JSON into configuration.cpp
alexdewar d9a8ab8
Add ConfigurationError class and use in a few places
alexdewar 100ffd7
Remove unused field
alexdewar d1cc571
Clean up and add extra error checking to config file loading
alexdewar f480522
Remove superfluous helper functions
alexdewar 2597d32
Fix: Seemingly MSVC isn't happy about casting filesystem::paths to st…
alexdewar 9a1188c
Move some functions out of configuration.cpp to program.cpp
alexdewar 3efc2be
Put functionality related to command-line args in own cpp file
alexdewar 0d1f803
Move some parsing functions into configuration_parsing.cpp
alexdewar 3d72290
Put configuration parsing helper function defs into own header
alexdewar edc505d
Rename some functions and mark noexcept
alexdewar 04eab1e
Remove unused API version functionality
alexdewar a7ee4b5
Move version.h.in to src/HealthGPS
alexdewar 7ff7cc1
Build most console functionality as lib for testability
alexdewar c6306a1
Spelling
alexdewar 3934a7f
Add JSON (de)serialisation code for Interval<> class
alexdewar 3111772
Make SettingsInfo.age_range an IntegerInterval
alexdewar 35b93a2
Base OptionalRange on Interval class and rename
alexdewar 9557033
Add tests for some config code
alexdewar efd97bf
Merge remote-tracking branch 'origin/main' into sanitise_input_data
alexdewar 9f5eaec
Fix typo
alexdewar 6180f36
Allow for parsing JSON objects to any map type with Identifier as key
alexdewar babf513
Use default comparators cf. custom operator==
alexdewar f5f55f4
Add test for load_interventions
alexdewar 3191348
Make active intervention into an optional type
alexdewar 4c9ee02
Add final tests for config code
alexdewar 5625e26
MSVC resources file should live in HealthGPS.Console/
alexdewar d4f531e
Remove unused var
alexdewar d5c2aa7
Use different get_to overload in a couple of places
alexdewar d19cdd8
Explicitly convert to string to appease MSVC
alexdewar c1b6636
Fix: Appease g++ v11 by not using structured bindings
alexdewar 9d9801d
Fix narrowing warning on MSVC
alexdewar 7deec87
Fix: Interval class's members should be default-initialised
alexdewar aafadfb
Fix/suppress bugprone-exception-escape clang-tidy warning
alexdewar 6f19143
Fix: Use absolute cf. weakly canonical paths in rebase_valid_path()
alexdewar cbd23d7
Fix docstring warnings and rename a couple of args
alexdewar a592a1b
Add docstrings to poco.h
alexdewar 26f1b31
Remove unused function definition
alexdewar 6438589
Add file-level documentation for several headers
alexdewar 0157f76
range parameter doesn't need a default value
alexdewar c9e3b2d
Revert "range parameter doesn't need a default value"
jamesturner246 72d6519
Remove special-casing for std::optional<Interval<T>>
alexdewar 00fc19f
Merge branch 'main' into sanitise_input_data
alexdewar File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Remove unused function definition
commit 26f1b3122e9f8b99181e89029cd82033919b314b
There are no files selected for viewing
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
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like this header is more for risk factor model JSON parsing stuff, right? Was the plan to have a separate header for the parsing interfaces of each model type? Maybe renaming this something like
configuration_parsing_risk_factor.h
would clarify this?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good idea. The reason I put these bits into their own header was really just because it felt like
configuration_parsing.h
was getting a bit cluttered and these functions only needed to be exported so we could access them for the tests.