Skip to content

feat: replace user ID obfuscation with SHA-256 hashing for enhanced security#1516

Open
polatengin wants to merge 1 commit into
mainfrom
polatengin/improving-obfuscation-logic
Open

feat: replace user ID obfuscation with SHA-256 hashing for enhanced security#1516
polatengin wants to merge 1 commit into
mainfrom
polatengin/improving-obfuscation-logic

Conversation

@polatengin
Copy link
Copy Markdown
Collaborator

  • Implemented hashUserId function to hash user IDs with board-specific salt using SHA-256.
  • Updated all components and services to use hashed user IDs instead of obfuscated ones.
  • Modified tests to reflect changes in user ID handling.
  • Deprecated obfuscateUserId and deobfuscateUserId functions for backward compatibility.
  • Added new cryptoHelper functions for secure hashing and token generation.

…ecurity

- Implemented hashUserId function to hash user IDs with board-specific salt using SHA-256.
- Updated all components and services to use hashed user IDs instead of obfuscated ones.
- Modified tests to reflect changes in user ID handling.
- Deprecated obfuscateUserId and deobfuscateUserId functions for backward compatibility.
- Added new cryptoHelper functions for secure hashing and token generation.
@polatengin polatengin self-assigned this Jan 24, 2026
@polatengin polatengin requested a review from a team as a code owner January 24, 2026 02:10
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.98%. Comparing base (411a32c) to head (95b7d69).
⚠️ Report is 8 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##              main    #1516      +/-   ##
===========================================
- Coverage   100.00%   99.98%   -0.02%     
===========================================
  Files           59       60       +1     
  Lines         4999     5030      +31     
  Branches      1284     1292       +8     
===========================================
+ Hits          4999     5029      +30     
- Misses           0        1       +1     
Files with missing lines Coverage Δ
src/frontend/components/boardSummaryTable.tsx 99.47% <100.00%> (-0.53%) ⬇️
...rontend/components/effectivenessMeasurementRow.tsx 100.00% <100.00%> (ø)
src/frontend/components/feedbackBoardContainer.tsx 100.00% <100.00%> (ø)
src/frontend/components/feedbackItem.tsx 100.00% <100.00%> (ø)
src/frontend/dal/itemDataService.tsx 100.00% <100.00%> (ø)
src/frontend/utilities/cryptoHelper.tsx 100.00% <100.00%> (ø)
src/frontend/utilities/userIdentityHelper.tsx 100.00% <100.00%> (ø)
Components Coverage Δ
frontend/components 99.97% <100.00%> (-0.03%) ⬇️
frontend/dal 100.00% <100.00%> (ø)
frontend/interfaces 100.00% <ø> (ø)
frontend/utilities 100.00% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@codecov-commenter
Copy link
Copy Markdown

Bundle Report

Changes will increase total bundle size by 468 bytes (0.04%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
retrospective-extension-webpack-bundle-array-push 1.17MB 468 bytes (0.04%) ⬆️

Affected Assets, Files, and Routes:

view changes for bundle: retrospective-extension-webpack-bundle-array-push

Assets Changed:

Asset Name Size Change Total Size Change (%)
./reflect-bundle.js 468 bytes 1.17MB 0.04%

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants