Skip to content

Axios create url bug #2290

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Sep 6, 2019
Merged

Conversation

dhrubesh
Copy link
Contributor

When using the request() method on an instance, the request would be successful even if the instance's config has url specified, but not baseUrl.

@dhrubesh
Copy link
Contributor Author

dhrubesh commented Jul 17, 2019

This PR solves #2234 .

G07cha pushed a commit to fed135/soixa that referenced this pull request Aug 10, 2019
Squashed from axios#2290:

fixing url issue while instance request

added spacing --eslint

added test cases

removed unexpected cases after updating the code
@felipewmartins
Copy link
Collaborator

Hi @dhrubesh, are you can update your local branch?

@felipewmartins felipewmartins merged commit 89bd3ab into axios:master Sep 6, 2019
felipewmartins added a commit to felipewmartins/axios that referenced this pull request Sep 13, 2019
genie-youn pushed a commit to genie-youn/axios that referenced this pull request Sep 27, 2019
* Fix axios#2234 

* added spacing --eslint

* added test cases

* removed unexpected cases after updating the code
genie-youn pushed a commit to genie-youn/axios that referenced this pull request Sep 27, 2019
@axios axios locked and limited conversation to collaborators May 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy