DynamoDB #6
Merged
DynamoDB #6
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
This commit introduces a new `DynamoDBStore` implementation to handle user credentials and data, replacing the in-memory user management. Key changes include creating and fetching users from DynamoDB, storing passkeys, and revising session management for WebAuthn operations. Additionally, it updates dependencies, improves error handling, and adjusts the state structure to align with the new backend integration.
Log and return detailed errors for DynamoDB operations in `db.rs` to improve debugging and resilience. Adjust `Cargo.toml` to lower optimization level and disable LTO for faster builds. Add missing deployment and DynamoDB setup instructions in the README, and include debug prints in `authn.rs` for better troubleshooting.
Add `println!` statements to log key actions in the code, such as user creation, query operations, and parsing. Updated the README to fix endpoint URLs and include proper command examples for manual testing setups.
Changed the WebAuthn RP ID and origin to "arkavo.net" to align with the updated domain. Also added a simpler `curl` example in the README for easier authentication testing.
Enhanced error handling in user creation, query, and registration workflows with detailed distinctions for missing tables and unknown errors. Added retries for database operations, input validations, and detailed logging to improve debugging and resiliency. Updated WebAuthn flows for better session state management and error reporting.
Introduce improved logging, validation, and error handling during credential registration and updates in the database. Ensure duplicate credentials are detected and prevent additions for nonexistent users, enhancing reliability and traceability.
Improved DynamoDB credential parsing by switching to list-based handling for better error resilience and efficiency. Updated environment variables in README to reflect production-ready DynamoDB tables. Minor logging enhancements and error handling improvements added for better debugging.
|
arkavo-com
changed the title
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.
DynamoDB Integration for User Management
DynamoDBStoreimplementation to replace in-memory user management.db.rsandauthn.rswith improved logging for key operations such as user creation and query processing.Credential Handling Improvements
WebAuthn Updates
Build and Deployment Updates
Cargo.toml:curlexamples for authentication testing.README Enhancements
Miscellaneous Improvements