UAC-Scripts(Bug Fixes and Upgrades)_S224373871_T3_2025#87
Open
VishalAbiman wants to merge 4 commits intoRedback-Operations:mainfrom
Open
UAC-Scripts(Bug Fixes and Upgrades)_S224373871_T3_2025#87VishalAbiman wants to merge 4 commits intoRedback-Operations:mainfrom
VishalAbiman wants to merge 4 commits intoRedback-Operations:mainfrom
Conversation
🔒 OWASP Scanner ResultsNo vulnerabilities detected. 🔒 OWASP Scanner Results for
|
🔒 Security Scan Results✅ No critical security issues detected. The code has passed all critical security checks. |
This file contains hidden or 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
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.
UAC Scripts Security & Functionality Overhaul
Project: SIT374 Capstone - User Access Control Scripts Improvement
Developer: Vishal Abiman (s224373871)
Period: Trimester 3, 2025
Key Improvements Made:
🔒 Security Fixes:
Critical Vulnerability: Fixed user overwriting bug in bulk-user-group-manager.sh that could accidentally delete existing accounts
Input Validation: Enhanced Y/N prompt validation across all scripts to prevent invalid inputs
Permission Handling: Resolved CSV file permission issues for secure credential logging
🐛 Bug Fixes:
Syntax Correction: Fixed critical array_minus() function error in start-of-tri-cleanup.sh that prevented script execution
Logic Flaws: Removed redundant project access questions that confused users
Error Handling: Improved error messages and recovery mechanisms
📝 Documentation & Quality:
Added comprehensive inline comments explaining security decisions
Created complete installation and usage guides
Developed testing framework with validation procedures
Updated README with detailed security considerations
🛡️ Compliance:
Ensured scripts meet ASD Essential Eight Maturity Level 1 standards
Implemented least privilege principles in group management
Added audit trails for all user operations
Technical Impact:
Before: Scripts had security vulnerabilities and usability issues
After: Production-ready tools with proper security controls
Tested: Validated on Ubuntu 22.04 with 50+ test cases
Result: Three fully functional, secure UAC scripts for educational environments
Repository Contains: All three fixed scripts (bulk-user-group-manager.sh, group-manager.sh, start-of-tri-cleanup.sh) with complete documentation and testing guides.