Skip to content
This repository was archived by the owner on May 1, 2020. It is now read-only.

[0.0.42] no longer supported by proxies list of ionic.config.json #346

Closed
konrin opened this issue Nov 10, 2016 · 12 comments
Closed

[0.0.42] no longer supported by proxies list of ionic.config.json #346

konrin opened this issue Nov 10, 2016 · 12 comments
Assignees

Comments

@konrin
Copy link

konrin commented Nov 10, 2016

Short description of the problem:

Stopped working proxy list of ionic.config.json. In consequence of this, stopped working xhr the application.

What behavior are you expecting?

Working proxy list of ionic.config.json.

Which @ionic/app-scripts version are you using?
0.0.42

Cordova CLI: 6.4.0
Ionic CLI Version: 2.1.7
Ionic App Lib Version: 2.1.4
ios-deploy version: 1.9.0
ios-sim version: 5.0.10
OS: macOS Sierra
Node Version: v7.0.0
Xcode version: Xcode 8.1 Build version 8B62

@peterpeterparker
Copy link

Same here!

GET http://localhost:8100/....404 (Not Found)

@MELAS007
Copy link

SAme here!

ionic serve --l doesn't work!

@Manduro
Copy link
Contributor

Manduro commented Nov 10, 2016

Working for me, but it's not for a co-worker using the very same project and up to date dependencies. Nope. It is working with [email protected] and [email protected] though.

@MELAS007
Copy link

MELAS007 commented Nov 10, 2016

@Manduro - How did you do?

@cambieri
Copy link

I'm using ionic-app-scripts version 0.0.43 and it seems to me that the problem appears only when there is more then one entry in the "proxies" array (ionic.config.json file). In that case, only the first entry works correctly, while the others are ignored.
Am I right? Will it takes long to fix this bug?
Thanks for the great work.

@jthoms1
Copy link
Contributor

jthoms1 commented Nov 14, 2016

Proxies should be supported in the latest release of ionic-app-scripts. I was unaware of the issue with it only using the first entry. I will test this out and provide my results today. Thanks!

@cambieri
Copy link

I'm very glad to help. Have you found the bug?
Please let me know if I can be of any further help.

@cambieri
Copy link

I can confirm that the fix proposed by vaskou works perfectly.
Will it be merged in the next release?
Thanks.

@cambieri
Copy link

Any news about this? The fix is still not presente in version 0.0.45.
Is it possible to merge the vaskou fix in the next release?
Thanks.

@jthoms1
Copy link
Contributor

jthoms1 commented Nov 18, 2016

Looks like @vaskou fix was merged. I will close this once it is released.

@cambieri
Copy link

Great news! Thank you.
Greetings.

@peterpeterparker
Copy link

Solved in 0.0.46, thx a lot @jthoms1 !

@jthoms1 jthoms1 closed this as completed Dec 21, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants