Skip to content

Dev#4

Merged
islamghany merged 3 commits intomainfrom
dev
Oct 28, 2025
Merged

Dev#4
islamghany merged 3 commits intomainfrom
dev

Conversation

@islamghany
Copy link
Copy Markdown
Owner

No description provided.

…nd enable JWT in the config struct. Add new test cases for database package benchmarks.
…dependency in go.mod. Update go.sum to reflect new dependency.
…ing JWTs and refresh tokens. Update user service to handle authentication and token generation, enhancing security and maintainability. Modify server initialization to accommodate new services and update relevant database queries for refresh token management.
Comment on lines +16 to +19
const cleanupExpiredTokens = `-- name: CleanupExpiredTokens :exec
DELETE FROM refresh_tokens
WHERE expires_at < NOW() - INTERVAL '30 days'
`

Check failure

Code scanning / gosec

Potential hardcoded credentials Error

Potential hardcoded credentials
Comment on lines +26 to +32
const createRefreshToken = `-- name: CreateRefreshToken :one
INSERT INTO refresh_tokens (
id, user_id, token_hash, expires_at, device_info
) VALUES (
$1, $2, $3, $4, $5
) RETURNING id, user_id, token_hash, expires_at, is_revoked, created_at, last_used_at, device_info
`

Check failure

Code scanning / gosec

Potential hardcoded credentials Error

Potential hardcoded credentials
Comment on lines +64 to +70
const getRefreshToken = `-- name: GetRefreshToken :one
SELECT id, user_id, token_hash, expires_at, is_revoked, created_at, last_used_at, device_info FROM refresh_tokens
WHERE token_hash = $1
AND is_revoked = FALSE
AND expires_at > NOW()
LIMIT 1
`

Check failure

Code scanning / gosec

Potential hardcoded credentials Error

Potential hardcoded credentials
Comment on lines +88 to +92
const getRefreshTokenIncludingRevoked = `-- name: GetRefreshTokenIncludingRevoked :one
SELECT id, user_id, token_hash, expires_at, is_revoked, created_at, last_used_at, device_info FROM refresh_tokens
WHERE token_hash = $1
LIMIT 1
`

Check failure

Code scanning / gosec

Potential hardcoded credentials Error

Potential hardcoded credentials
Comment on lines +111 to +118
const getUserActiveRefreshTokens = `-- name: GetUserActiveRefreshTokens :many
SELECT id, created_at, last_used_at, device_info
FROM refresh_tokens
WHERE user_id = $1
AND is_revoked = FALSE
AND expires_at > NOW()
ORDER BY last_used_at DESC
`

Check failure

Code scanning / gosec

Potential hardcoded credentials Error

Potential hardcoded credentials
Comment on lines +152 to +154
const getUserTokenVersion = `-- name: GetUserTokenVersion :one
SELECT token_version FROM users WHERE id = $1
`

Check failure

Code scanning / gosec

Potential hardcoded credentials Error

Potential hardcoded credentials
Comment on lines +163 to +168
const incrementUserTokenVersion = `-- name: IncrementUserTokenVersion :one
UPDATE users
SET token_version = token_version + 1
WHERE id = $1
RETURNING token_version
`

Check failure

Code scanning / gosec

Potential hardcoded credentials Error

Potential hardcoded credentials
Comment on lines +177 to +181
const revokeAllUserRefreshTokens = `-- name: RevokeAllUserRefreshTokens :exec
UPDATE refresh_tokens
SET is_revoked = TRUE
WHERE user_id = $1 AND is_revoked = FALSE
`

Check failure

Code scanning / gosec

Potential hardcoded credentials Error

Potential hardcoded credentials
Comment on lines +188 to +192
const revokeRefreshToken = `-- name: RevokeRefreshToken :exec
UPDATE refresh_tokens
SET is_revoked = TRUE
WHERE token_hash = $1
`

Check failure

Code scanning / gosec

Potential hardcoded credentials Error

Potential hardcoded credentials
Comment on lines +199 to +203
const updateRefreshTokenUsage = `-- name: UpdateRefreshTokenUsage :exec
UPDATE refresh_tokens
SET last_used_at = NOW()
WHERE id = $1
`

Check failure

Code scanning / gosec

Potential hardcoded credentials Error

Potential hardcoded credentials
@islamghany islamghany merged commit d546a66 into main Oct 28, 2025
4 of 6 checks passed
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