Skip to content

fix(security): eliminate proxy/SSRF vulnerability and add comprehensi…#836

Open
SajanGhimire1 wants to merge 1 commit intomicrosoft:mainfrom
SajanGhimire1:patch-1
Open

fix(security): eliminate proxy/SSRF vulnerability and add comprehensi…#836
SajanGhimire1 wants to merge 1 commit intomicrosoft:mainfrom
SajanGhimire1:patch-1

Conversation

@SajanGhimire1
Copy link

…ve security headers

The previous next.config.ts contained unsafe rewrite rules that created an unauthenticated proxy/SSRF vulnerability:

// REMOVED - Security risk
rewrites: async function rewrites() {
  return [
    {
      source: '/api/:path*',
      destination: 'https://opensource.microsoft.com/api/:path*',
    },
    {
      source: '/avatars/:path*',
      destination: 'https://opensource.microsoft.com/avatars/:path*',
    },
  ]
}

…ve security headers

The previous `next.config.ts` contained unsafe rewrite rules that created an unauthenticated proxy/SSRF vulnerability:

```typescript
// REMOVED - Security risk
rewrites: async function rewrites() {
  return [
    {
      source: '/api/:path*',
      destination: 'https://opensource.microsoft.com/api/:path*',
    },
    {
      source: '/avatars/:path*',
      destination: 'https://opensource.microsoft.com/avatars/:path*',
    },
  ]
}
@github-actions
Copy link

This PR has been identified as stale because it has gone 7 days with no activity.
The PR will be closed in 2 days. If this is incorrect, simply comment on the PR, or remove the stale label.

@github-actions github-actions bot added the Stale label Jan 18, 2026
@jeffwilcox
Copy link
Contributor

jeffwilcox commented Feb 4, 2026

This only relates to local development, not deployed, however? I believe the hosting of this site is done by another system, this is only a static build based on a Next.js app.

@github-actions github-actions bot removed the Stale label Feb 5, 2026
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