다음은 실행 중인 예입니다. https://runkit.com/philipbjorge/axios-regression

세션 기반 인증 토큰도 병합되지 않는 유사한 문제가 있습니다.

수정은 params 를 속성의 deepMergeProperties 섹션으로 옮기는 것만큼 간단합니다. 차이점: https://github.com/axios/axios/compare/8d0b92b2678d96770304dd767cd05a59d37f12cf...zackseuberling :zs-fix-merge-params

@zackseuberling 이에 대한 풀 리퀘스트를 입력할 수 있습니까?

같은 문제가 있습니다.

@jasonsaayman 테스트가 이제 #2196에서 (올바른 동작이라고 가정하기 위해) 통과하고 있습니다.

곧 출시되기를 바랍니다. 문제는 나를 꽤 짜증나게 만들었습니다.

같은 문제가 있습니다. 언제 수정될 예정인가요?

@wphestiraid도 마찬가지입니다 😬

0.18.1 다운그레이드하면 해결 방법으로 해결되었습니다! 보안 취약점이 수정되었습니다.

@serranoarevalo 저는 0.18.1 사용할 수 있다고 생각합니다 :smile:

또한이 일어날 것으로 보인다 method 가 단지에 지정된 않다면, create 호출이 아니라 request . 그것을 알아내는 데 하루 종일 걸렸습니다. 확실히 획기적인 변화였습니다. (이것은 별도의 문제여야 합니까, 아니면 동일한 일반적인 문제의 일부입니까?)

여기서도 같은 문제

이것에 대한 소식이 있습니까? 언제 고쳐질까?

이것에 대한 소식이 있습니까? 언제 고쳐질까?

당분간 v0.18.1 를 사용하십시오. ✈️

그것이 내가 사용하고 있는 것이지만 보안 문제로 인해 이 문제가 해결되면 v0.19.0 로 업그레이드해야 합니다.

그것이 내가 사용하고 있는 것이지만 보안 문제로 인해 이 문제가 해결되면 v0.19.0 로 업그레이드해야 합니다.

어떤 종류의 보안 문제? v0.18.0 에 보안 문제가 있었지만 v0.18.1 는 괜찮습니다.

NPM은 여전히 ​​감사 중에 0.18.1을 취약한 것으로 표시하고 있습니다(적청어일 가능성이 있음).

저처럼 axios 0.19 를 사용해야 하는 경우 매개변수를 이와 같이 관리할 수 있으며 현재로서는 해결 방법 으로 잘 작동합니다.

// Pass to create only these params
const instance = axios.create({
  headers,
  baseURL,
  timeout,
});

// Pass others to the instance request
instance.request({
  method,
  url,
  data,
  ...options,
});

정식 수정을 기다리며 😭

찬성. 이 버그를 쫓는 데 오후를 보냈고 문제를 작성하기 위해 여기에 왔습니다.

수정 사항을 제공하기 위한 진행 상황이 있습니까?

https://github.com/axios/axios/blob/v0.19.0/lib/core/mergeConfig.js#L18 -L22

module.exports = function mergeConfig(config1, config2) {
  // eslint-disable-next-line no-param-reassign
  config2 = config2 || {};
  var config = {};

  utils.forEach(['url', 'method', 'params', 'data'], function valueFromConfig2(prop) {
    if (typeof config2[prop] !== 'undefined') {
      config[prop] = config2[prop];
    }
  });

v0.19.0 코드는 mergeConfig(axios_instance.defaults, config)
요청 config.params만 사용됩니다.

부 버전이 이전 버전과 호환되지 않는 이유는 무엇입니까?
이것에 대한 테스트가 없습니까?
물어봐야 해요.

@magicdawn 제 생각에는 그들이 망친 것 같습니다. 나는 그들이 수정 사항을 병합하기 위해 무엇을 기다리고 있는지 이해하지 못합니다. @emilyemorehouse 해주세요 https://github.com/axios/axios/pull/2196 🙏!

소식이 있나요? 동일한 오류가 발생하여 최신 버전으로 업그레이드했습니다.

문제는 이것이 곧 해결된다는 것입니다!!

이 문제를 찾는 데 시간이 좀 걸렸습니다... 짜증나네요. 수정도 기다리고 있습니다.

해킹일 수 있지만 생성된 axios 인스턴스에 대해 get 메소드를 패치할 수도 있습니다.

const client = axios.create({
  baseURL: YOUR_URL,
})
client.originalGet = client.get
client.get = (url, { params, ...config }) =>
  client.originalGet(url, {
    ...config,
    params: {
      key: YOUR_API_KEY,
      ...params,
    },
  })


// use it like normal
client.get('/something', { params: paramsToMerge } )

인스턴스 생성을 래핑하고 다음과 같이 모든 메서드를 패치했습니다.

export const createInstance = (config?: AxiosRequestConfig): AxiosInstance => {
  const instance = axios.create(config);

  // Patch methods to merge default params into each request.
  // https://github.com/axios/axios/issues/2190
  for (const [method, configIndex] of methodsToPatch as Array<[keyof AxiosInstance, number]>) {
    const orig = instance[method] as Function;
    // @ts-ignore
    instance[method] = (...args: any[]) => {
      if (!args[configIndex]) {
        args[configIndex] = {};
      }
      args[configIndex].params = {
        ...(instance.defaults.params || {}),
        ...(args[configIndex].params || {}),
      };
      return orig.apply(instance, args);
    };
  }

  return instance;
};

const methodsToPatch = [
  ['getUri', 0],
  ['request', 0],
  ['get', 1],
  ['delete', 1],
  ['head', 1],
  ['post', 2],
  ['put', 2],
  ['patch', 2],
];

해결 방법을 찾는 모든 사람을 위해 내 것을 사용할 수 있습니다(모든 사람에게 적용되지 않을 수 있음).

npm i qs

axios.interceptors.request.use(config => {
    config.paramsSerializer = params => qs.stringify({ ...params, myValue: 'foo' });
    return config;
}

@janpantel 궁금한데 매개변수만 수정하면 되지 않나요?

axios.interceptors.request.use(config => {
    config.params.myValue = 'foo';
    return config;
}

@janpantel 덕분에 그렇게 작동한다고 생각하고 다른 대안보다 깨끗합니다.

우리는 매우 유사한 오류가 있습니다.

사실 axios.create() params 인스턴스를 지정할 수도 없습니다.
그 동안 우리는 요청 인터셉터로 돌아갔습니다 ...

저에게도 마찬가지입니다. 오늘 날 미치게 해! 모두 잘 작동했습니다. 그런 다음 갑자기 인증 문제와 CORS 문제가 발생했습니다! 야...

나는 v18에서 v19로 업데이트 한 다음이 버그를 상속 받았다는 것을 알아 냈습니다. 인스턴스 정의에서 정의한 '기본' 매개변수(구식 authkey 매개변수)가 나중에 새 매개변수로 해당 인스턴스를 호출했을 때 병합되지 않은 것으로 보입니다. 새 매개변수는 기본값을 죽였습니다.

버전 18로 다운 그레이드하면 해결되었습니다. 그들이 곧 수정하기를 바랍니다. 저는 모든 모듈이 최신 상태인 것을 좋아합니다! :NS

이제 노드 및 브라우저에서 umi-request를 사용하도록 설정합니다.
여태까지는 그런대로 잘됐다. 😄

같은 문제가 2020년 이후로 여전히 종료됩니다.
설계된 대로 실행됩니까? 아니면 언제 이 문제를 해결할 수 있습니까?

Axios 버전: 0.19.0

유용한 라이브러리를 제공해주셔서 감사합니다!

v0.19.1 에는 여전히 이 버그가 있습니다 😞

이 문제를 이미 수정하십시오. 이것은 정말 짜증납니다. 이 버그를 파악하는 데 15분이 걸렸습니다.

[email protected] 사용하는 것과 동일한 문제입니다. 인터셉터를 해결 방법으로 사용합니다.

[email protected]에서 여전히 동일한 문제가

npm 설치 --save [email protected]

이 문제도 있습니다. 고쳐주세요

일종의 끝없는 드라마

인스턴스 매개변수가 호출 매개변수와 병합되지 않는 것도 아닙니다. 인스턴스 매개변수는 요청에 대한 매개변수에 전혀 들어가지 않습니다. 인스턴스를 만들고 params를 정의하면 get 요청 중 하나에서 찾을 수 없습니다.

모든 get 호출에서 매개변수를 정의하는 것보다 다운그레이드가 더 쉬운 옵션이었습니다.

이 문제가 종료된 이유는 무엇입니까? @renestalder가 언급했듯이 마지막 버전에서는 create 메서드를 다소 무의미하게 만드는 모든 요청에 ​​인스턴스 매개변수가 나타나지 않습니다.

#2656에 수정되어 종료되었지만 아직 버전이 없습니다.

@cheelahim이 제안한 것처럼 인터셉터도 해결 방법으로 사용했습니다.

axios.interceptors.request.use(config => {
  config.params. = {
    myValue1: 'foo',
    myValue2: 'bar',
    ...config.params,        
  }
    return config;
}

그러나 여전히 이 버그를 수정하기 위한 새 버전을 기다리고 있습니다.

Axios get 및 매개변수

      const params = {
        name,
        id,
        pageIndex,
        pageSize,
      };
      const url = `/api/poi/query`;
      axios
      // .get(url, params)
      .get(url, {
        params,
      })
    .then(json => {
        // do something
      })
     .catch(err => console.log(`error`, err));

https://stackoverflow.com/questions/40947650/axios-get-in-url-works-but-with-second-parameter-as-object-it-doesnt

와! 거의 1년 후!

버그는 여전히 존재합니다! 믿을 수 없다......

수정 사항은 #2656에 병합되었지만 아직 수정되지 않은 버전입니다.

그냥 0.18.0으로 다운그레이드

폐쇄된 이유는 무엇입니까?

불행히도 fetch 은 진행 이벤트 또는 취소 지원이 필요한 즉시 종료됩니다.

버전을 ^0.18.1 고정하기만 하면 됩니다.

@techouse 가 대안으로 이동하는 것이 더 나은 솔루션입니다.

저는 항상 axios를 사용하는 것을 좋아했습니다. 그러나 이 수정 사항이 오랫동안 존재했으며 아무도 이를 버전에 할당할 수 없다는 생각은 농담일 뿐이므로 계속 진행해야 합니다. 또는 실제로 제품을 발전시키고자 하는 누군가를 관리하도록 하십시오.

존 알베르기니
기술 이사
[email protected]
영형. 802-323-4558 | 씨. 802-323-4558 | www.legacyresearch.com http://www.legacyresearch.com

보낸 사람: Evgeny [email protected]
보낸 날짜: 2020년 4월 23일 목요일 오전 10:59:32
받는 사람: axios /
참조: Jon Alberghini [email protected] ; 댓글 [email protected]
제목: Re: [axios/axios] 요청 매개변수가 인스턴스 매개변수와 병합되지 않습니다(#2190).

@techouse https://github.com/techouse 대안으로 이동하는 것이 더 나은 솔루션입니다.

—
당신이 댓글을 달았기 때문에 이것을 받는 것입니다.
이 이메일에 직접 답장하거나 GitHub https://github.com/axios/axios/issues/2190#issuecomment-618445756 에서 확인하거나 https://github.com/notifications/unsubscribe-auth/AF6OZWIPREAQP5OGY437X3ROBJVJANFSM4HRPA를 구독 취소

수정 사항 이 병합되어 v0.20.0에서 릴리스될 예정이기 때문에 종료된 것 같습니다. 여기를 참조

여기에도 같은 문제가 있습니다. 곧 수정되기를 바랍니다!

+1. 일단 다운그레이드. 으음.

@khastation @raphaelbadawi @jonathanalberghini @techouse @AndyOGo

대안:

• 페치 및 폴리필 whatwg-fetch
• 갖다
• 키
• 노드 가져오기

@mahnunchik
불행히도 fetch 는 진행 이벤트 또는 취소 지원이 필요한 즉시 종료됩니다.

@AndyOGo https://javascript.info/fetch-progress 와 같이 구현할 수 있습니다.

키 매개변수를 분리하기 위해 일부 분해를 사용하여 탈출구를 찾았습니다. 임시 코드는 확실하지만 이미 병합이 완료되고 0.20.0이 진행 중이므로 정말 자신합니다. fetch() 경험이 있더라도 Axios는 여전히 갈 길입니다. Axios에서는 일반적으로 상황이 매우 간단합니다. ;-)

@mahnunchik 감사합니다
다운로드 및 업로드 에 대한 진행 이벤트 에 대해 이야기하고 있지만!

질문 감사합니다 @bdrtsky
주로 다음과 같은 이유로:

• fetch 는 XMLHttpRequest 비해 여러 회귀를 지정합니다.
• 대규모 서비스를 구축하는 즉시 취소 가 중요해집니다.
• 파일 업로더를 구축했으므로 보다 정교한 진행 처리가 필요합니다.

수정 사항이 이미 0.20 릴리스에 할당되어 있기 때문에 종료되었습니다. 우리는 현재 0.20의 지점에서 회귀를 수정하고 안정적인 Axios를 다시 출시하기 위해 작업하고 있습니다.

이것은 진행중인 작업이며 업데이트를 위해 0.20 프로젝트를 확인하십시오.

0.19.3 패치 버전에 넣어도 되나요? 동일한 마이너 릴리스에서 수정되지 않으면 결함을 닫을 수 없습니다 ...

나는 다시 열었지만 우리는 0.19.3을 출시하지 않을 것입니다. 0.18.0에서 0.19.2로의 회귀를 모두 제거하는 0.20 릴리스에서 안정성을 얻고 싶습니다.

가능한 유일한 해결책은 다운그레이드하는 것입니다.

이 애매한 0.20 릴리스를 언제 기대할 수 있습니까? 그동안 사용할 수 있는 베타 버전이 있습니까?

@adamreisnz 저는 0.20 릴리스에서 작업하고 있지만 여전히 약 60개의 이상한 pull 요청이 있습니다(120개의 홀수에서 감소). 일부는 이러한 pull 요청이 여전히 관련이 있는지 확인하고 확인하는 데 매우 오래되고 다양한 작업이 있습니다. 주요 변경 사항을 제시하지 않고, 테스트를 하고, 컴파일된 파일을 포함하지 않는 등의 작업은 노동입니다. 기껏해야 7월까지 사전 출시될 수 있도록 노력하고 있습니다. v1에서 작업하기 전에 바로 얻을 수 있기를 원하므로 이 릴리스를 테스트하는 데 도움이 되었으면 합니다.

성가신 문제입니다. 해결될 수 있기를 바라며 오랫동안 유지됩니다.

v1 작업을 하기 전에 바로 확인하고 싶습니다.

0.x 에는 1.0.0 이전에 추가 릴리스를 만들 수 있는 충분한 공간이 있습니다. 다음 릴리스에 모든 것을 집어넣을 필요는 없습니다. 사실, 잠재적인 새로운 버그를 추적하기가 더 어려워질 것이라고 주장할 수 있습니다.

@markcarver 물론 가능하지만 Axios는 현재 v1이 없는 시험판으로 간주됩니다. 대부분의 사람들은 Axios가 공개 API를 출시하지 않은 것으로 보입니다 . v0.20.0에서는 안정성을 가져오고 0.19.x 분기에서 발생하는 대부분의 회귀를 수정하고 싶습니다. 이 분기에 몇 가지 문제가 있을 뿐만 아니라 쉽게 승리할 수 있는 오류에 대한 작업입니다.

또한 동시에 pull 요청의 백로그를 지워야 합니다. 그 중 일부는 2017년부터 있었던 120개의 이상한 풀 리퀘스트를 처리하는 것은 매우 어렵습니다. 나는 우리가 이 문제에 도달할 것이라고 약속합니다. 저는 아무데도 가지 않고 가능한 한 Axios를 지원하고 작업할 것입니다.

감사 해요

우리가 v0.18에서 Axios를 다운그레이드하고 잠그거나 자체(유지보수되지 않은) 버전을 롤링하고 수정 사항을 적용하지 않는 한 특정 플러그인(예: axios 캐시 어댑터)을 사용할 수 없기 때문에 좌절감이 많이 쌓이는 것 같아요. .

이것은 꽤 몇 달 동안 사실이었고 모든 사람들이 최소한 몇 가지 수정 사항이 릴리스되기를 열망하고 있다고 생각합니다.

최소한 최신 코드베이스를 사용할 수 있도록 현재 적용된 수정 사항 중 일부를 이식하고 0.19.x를 릴리스할 수 없습니까? 또 다른 120개의 풀 리퀘스트를 진행할 예정이라면 0.20 또는 v1이 나올 때까지 오랜 시간을 기다려야 할 것 같습니다.

가능하면 개인적으로 더 점진적인 접근 방식을 선택합니다.

@adamreisnz 나도 증분을 선택할 것이라고 확신하지만 0.18.x에서 0.19.x로 많은 부분이 깨졌고 아이디어는 0.20.0 릴리스를 그 릴리스에서 가장 시급한 것들로 안정되게 만드는 것이 었습니다. 60개의 풀 리퀘스트만 남았고 나머지는 이미 처리했고 그 60개 중 일부는 wip 또는 v1 태그가 지정되었습니다.

오랜 시간이 걸렸다는 것을 알고 있지만 이번 기회에 바로잡아 더 많은 일을 일으키고 깨뜨리는 것이 아니라 이 시점에서 대체로 가능한 일을 하고 싶습니다. 또한 빌드는 공개되기 전에 광범위한 테스트가 필요합니다. 좌절감을 느끼긴 하지만 안정적이지 않고 수많은 회귀를 수정하지 않으면 증분 수에 관계없이 릴리스할 수 있으며 여전히 200~개 이상의 문제와 60~개의 풀 리퀘스트가 남아 있습니다. 여전히 매우 부서진 Axios, 어쩌면 훨씬 더 부서질 수도 있습니다.

또한 저는 현재 릴리스를 제어하지 않으며 지원을 시작했을 때 회귀 및 가장 시급한 문제를 수정하고 이전에 작동했던 다른 것을 손상시키지 않는 빌드를 함께 만들겠다고 약속했습니다. 나는 이것을 꽤 진지하게 받아들이고 그렇게 하고 싶다.

그러나 도움이 되는 경우 보안 문제 등이 수정된 0.18.x 릴리스를 시도하고 얻을 수 있습니까?

그러나 도움이 되는 경우 보안 문제 등이 수정된 0.18.x 릴리스를 시도하고 얻을 수 있습니까?

정말 대단합니다.

추신: 많은 웹 프로젝트에 복잡한 패키지를 유지 관리하여 귀하가 하고 있는 노고에 감사드립니다. 😊

그러나 도움이 되는 경우 보안 문제 등이 수정된 0.18.x 릴리스를 시도하고 얻을 수 있습니까?

예, 동의합니다. 확실히 감사하겠습니다.

새로운 항목을 테스트하는 데 도움이 필요하면 기꺼이 그렇게 하겠다고 알려주세요.

@adamreisnz 이 분기를 테스트할 수 있습니까: 0.20.0-beta.1

예, 이번 주말에 우리 프로젝트 중 하나를 위해 해당 지점으로 전환하려고 합니다.

@jasonsaayman 지금까지 너무 좋았어, 0.20.0이 잘 작동하는 것 같다 👍

이제 문제를 종료할 수 있습니다.

이제 새 릴리스에서 제대로 작동한다는 피드백을 받은 후 종료합니다.