fix(fetch): encode parameters in source URL (#817)

* fix(fetch): encode parameters in source URL

* chore(temp): create openapi.json

* fix: do this instead

looks like ampersands are escaped by default, this looks like the move

* chore: test name

* revert: remove dummy file

* Revert "fix: do this instead"

This reverts commit 4b0075b.

__tests__/lib/fetch.test.ts

@@ -64,6 +64,31 @@ describe('#fetch()', () => {
         mock.done();
       });
 
+      it('should include source URL header with path that contains weird characters', async () => {
+        const key = 'API_KEY';
+
+        const mock = getAPIMock()
+          .get('/api/v1')
+          .basicAuth({ user: key })
+          .reply(200, function () {
+            return this.req.headers;
+          });
+
+        const headers = await readmeAPIFetch(
+          '/api/v1',
+          {
+            method: 'get',
+            headers: cleanHeaders(key),
+          },
+          { filePath: './📈 Dashboard & Metrics/openapi.json', fileType: 'path' }
+        ).then(handleRes);
+
+        expect(headers['x-readme-source-url'].shift()).toBe(
+          'https://github.com/octocat/Hello-World/blob/ffac537e6cbbf934b08745a378932722df287a53/%F0%9F%93%88%20Dashboard%20&%20Metrics/openapi.json'
+        );
+        mock.done();
+      });
+
       it('should include source URL header with relative path', async () => {
         const key = 'API_KEY';
 

src/lib/readmeAPIFetch.ts

@@ -164,7 +164,9 @@ export default async function readmeAPIFetch(
        */
       headers.set(
         'x-readme-source-url',
-        `${process.env.GITHUB_SERVER_URL}/${process.env.GITHUB_REPOSITORY}/blob/${process.env.GITHUB_SHA}/${filePath}`
+        encodeURI(
+          `${process.env.GITHUB_SERVER_URL}/${process.env.GITHUB_REPOSITORY}/blob/${process.env.GITHUB_SHA}/${filePath}`
+        )
       );
     }
   }