Rollbacking lowercasing paths since it breaks share links... (#271)

Author: jpreynat

src/lib/middleware.test.ts

@@ -118,15 +118,15 @@ describe('normalizeURL', () => {
  );
  });
 
- it('should convert uppercase characters in the path to lowercase', () => {
- expect(normalizeURL(new URL('https://docs.mycompany.com/Hello/My/pAge'))).toEqual(
- new URL('https://docs.mycompany.com/hello/my/page'),
- );
- });
+ // it('should convert uppercase characters in the path to lowercase', () => {
+ //  expect(normalizeURL(new URL('https://docs.mycompany.com/Hello/My/pAge'))).toEqual(
+ //  new URL('https://docs.mycompany.com/hello/my/page'),
+ //  );
+ // });
 
- it('should not affect uppercase characters in querystring parameters', () => {
- expect(normalizeURL(new URL('https://docs.mycompany.com/Hello/My/pAge?Q=MySearch'))).toEqual(
- new URL('https://docs.mycompany.com/hello/my/page?Q=MySearch'),
- );
- });
+ // it('should not affect uppercase characters in querystring parameters', () => {
+ //  expect(normalizeURL(new URL('https://docs.mycompany.com/Hello/My/pAge?Q=MySearch'))).toEqual(
+ //  new URL('https://docs.mycompany.com/hello/my/page?Q=MySearch'),
+ //  );
+ // });
 })

src/lib/middleware.ts

@@ -94,6 +94,7 @@ export function getURLLookupAlternatives(input: URL) {
  */
 export function normalizeURL(url: URL) {
  const result = new URL(url);
- result.pathname = url.pathname.replace(/\/{2,}/g, '/').replace(/\/$/, '').toLowerCase();
+ // TODO fix lowercasing only the pathname without share links...
+ result.pathname = url.pathname.replace(/\/{2,}/g, '/').replace(/\/$/, ''); // .toLowerCase();
  return result;
 }