Skip to content

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

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

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

This comment was marked as outdated.

@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
@github-actions github-actions bot added the Stale label Feb 12, 2026
@microsoft microsoft deleted a comment from github-actions bot Mar 6, 2026
@jeffwilcox jeffwilcox removed the Stale label Mar 6, 2026
@jeffwilcox
Copy link
Contributor

Closing as indicated.

@jeffwilcox jeffwilcox closed this Mar 6, 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