fix: fix Parsing Error & TypeError

[gahyuun]

📌 작업 내용

구현 내용 및 작업 했던 내역

closed #21
closed #22

• headers 객체 Record<string,string> 타입으로 지정
• baseURL과 url을 new URL을 통해 합치는 것이 아닌 url이 상대 경로일 경우에만 문자열 합치는 형식으로 변경
• isBodyInit 함수에서 data type을 확인할 때 data가 JSON 타입이면 typeof data === 'string' 조건문에서 true더라구요 하지만 조금 더 명시적이면 좋을 것 같아 isJSON 함수를 구현 후 조건문에 추가해줬습니다

🔊 도움이 필요한 부분

• baseURL이 지정되어 있지 않은 상태에서 '/user/sr'와 같은 상대 경로를 넣으면 Invalid URL 에러가 발생합니다. (그래서 URL 쪽 일부를 수정했습니다.) baseURL 지정이 안 되어 있을 때 해당 origin으로 요청을 보내도록 개선하면 좋을 것 같습니다. (Migration ax fetch myongji-graduate-next#149)
• 저번 PR에서 baseURL이 없는 상태에서 /user/sr 상대 경로로 요청을 보내면 InvalidURL Error가 아닌 origin으로 요청을 보내야 할 것 같다고 하셨는데, /user/sr로 보내도 어차피 Fetch API에서 InvalidError를 발생시키더라구요 fetch-ax에서 InvalidError를 발생시키지 않게 수정을 해야 할 것 같다고 생각하신 이유가 있을까요??
• 저도 Fetch API에서 InvalidError 를 발생시키는 걸 뒤늦게 확인해서 일단 구현은 된 상태이며 baseURL, url 둘다 있는데 만약 url이 절대경로라면 baseURL이 무시되는 기능도 추가했습니다.

TypeError: Failed to parse URL from sign-in
  [cause]: TypeError: Invalid URL

[seonghunYang]

• baseURL이 없고 절대 URL이 아닌 경우에는 fetch에서 Invalid URL 발생하도록 유도한건가여?

[gahyuun]

네넵 ! "/a" 이런 식으로 요청 했을때 fetch에서 InvalidURL Error를 발생시키더라구요 그래서 제가 생각하기엔 저희가 InvalidURL을 굳이 코드로 막을 필요가 없다고 느꼈습니다!

[gahyuun]

이 부분에 대해서 어떻게 생각하시나요??

[seonghunYang]

• 밑에 의견 남긴거랑 동일한 것 같은데 axios의 경우는 절대 경로가 아닌 상대 경로가 입력될 때 baseURL이 없으면 기본적으로 origin 요청을 보내고 있고, 제 사용 경험을 고려했을 때도 저는 필요한 기능이라고 생각이 들어요
• 다만, 이 PR이랑 별개의 작업으로 따로 처리하는게 좋을 것 같습니다. 따로 issue를 만들어 놓고 다음 PR에서 제가 처리해보도록 할게요!

[gahyuun]

아하 그러면 만약 아래와 같이 baseURL이 없고 상대 경로만 있을 때 클라이언트주소/a (ex. http://localhost:3000/a)로 요청을 보내는 게 맞을까요??
저는 처음에 baseURL이 없고 /a가 없는데 서버 도메인을 어떻게 알고 요청을 보내지? 라는 생각 때문에 이해를 한 번에 하지 못한 것 같아요 🥹

const response = await fetch-ax('/a', {
          headers: {},
          // @ts-ignore
        });

[seonghunYang]

네 맞습니다!

[seonghunYang]

• 저번 PR에서 baseURL이 없는 상태에서 /user/sr 상대 경로로 요청을 보내면 InvalidURL Error가 아닌 origin으로 요청을 보내야 할 것 같다고 하셨는데, /user/sr로 보내도 어차피 Fetch API에서 InvalidError를 발생시키더라구요 fetch-ax에서 InvalidError를 발생시키지 않게 수정을 해야 할 것 같다고 생각하신 이유가 있을까요??

axios의 경우, 절대 경로가 아닌 상대 경로가 입력될 때 baseURL이 없으면 기본적으로 origin을 요청으로 보냅니다. 따라서 개발 편의성을 위해 필요한 기능이라고 판단하여 제안드렸습니다.

[yougyung]

수고하셨습니다.
example폴더의 interceptor와 README도 변경사항이 반영되면 좋을 것 같아요!

[gahyuun]

수고하셨습니다. example폴더의 interceptor와 README도 변경사항이 반영되면 좋을 것 같아요!

헉 큰일날뻔했네요ㅠㅠ 말씀해주셔서 감사합니다
7ee068c