fix: Update URL resolution logic to handle search parameters on root path /?foo=bar (#78262)

## What?

This PR fixes and implements test for an URL resolution issue that
stripped the searchparam when `metadataBase` was defined and a search
param was set on the root path. This created a behavior where :

```
export const metadata: Metadata = {
  title: "Create Next App",
  description: "Generated by create next app",
  metadataBase: new URL("https://www.example.com"),
  alternates: {
    canonical: "/?foo=bar",
  },
};
```

would result in 

`<link rel="canonical" href="https://www.example.com">` 

instead of 

`<link rel="canonical" href="https://www.example.com/?foo=bar">`



## Related issues

- Fixes #65776
- Fixes #74689

Co-authored-by: Jiwon Choi <[email protected]>

Author: F7b5

packages/next/src/lib/metadata/resolvers/resolve-url.test.ts

@@ -62,6 +62,12 @@ describe('resolveAbsoluteUrlWithPathname', () => {
         'https://example.com/foo'
       )
     })
+
+    it('should resolve absolute internal url with query', () => {
+      expect(resolver('https://example.com/?foo=bar')).toBe(
+        'https://example.com/?foo=bar'
+      )
+    })
   })
 
   describe('trailingSlash is true', () => {
@@ -102,13 +108,20 @@ describe('resolveAbsoluteUrlWithPathname', () => {
       expect(resolver(new URL('https://example.com/foo?bar'))).toBe(
         'https://example.com/foo?bar'
       )
+      expect(resolver('https://example.com/?foo=bar')).toBe(
+        'https://example.com/?foo=bar'
+      )
     })
 
     it('should not add trailing slash to relative url with query', () => {
       expect(resolver('/foo?bar')).toBe('https://example.com/foo?bar')
       expect(resolver(new URL('/foo?bar', metadataBase))).toBe(
         'https://example.com/foo?bar'
       )
+      expect(resolver('/?foo=bar')).toBe('https://example.com/?foo=bar')
+      expect(resolver(new URL('/?foo=bar', metadataBase))).toBe(
+        'https://example.com/?foo=bar'
+      )
     })
 
     it('should not add trailing slash to relative url that matches file pattern', () => {

packages/next/src/lib/metadata/resolvers/resolve-url.ts

@@ -114,7 +114,10 @@ function resolveAbsoluteUrlWithPathname(
   if (typeof result === 'string') {
     resolvedUrl = result
   } else {
-    resolvedUrl = result.pathname === '/' ? result.origin : result.href
+    resolvedUrl =
+      result.pathname === '/' && result.searchParams.size === 0
+        ? result.origin
+        : result.href
   }
 
   // Add trailing slash if it's enabled for urls matches the condition