Comments (3)
Fix published in 5.0.3, thanks!
https://github.com/3lvis/Networking/releases/tag/5.0.3
Finding this project helpful? Consider supporting further development and support by becoming a sponsor:
👉 https://github.com/sponsors/3lvis
from networking.
Hello @mkll!
1.- I think your logic makes sense, somehow the URL session configuration should have been connected to the way caching is handled in Networking. Sadly, it wasn't done that way, instead caching is handled at the call level. Whenever you make a Networking call you can decide the caching level, the default is none.
If I were to make the change now of connecting URL session configuration with how caching is handled I'm worried it might lead to inconsistent behavior, where some people using ephemeral tried to do caching here and there and it wouldn't work.
If caching is none by default and you aren't using any caching I'm surprised why this crash is even happening. Either way, it's something we need to fix.
2.- The fatalError
is not necessary. I'll remove it. If we can't construct the URL in this method we should just escape instead of crashing.
3.- My answer here is similar to the 1st one, even though your logic makes sense I'm afraid it could cause unwanted behavior.
from networking.
@3lvis thanks for the clarification.
Imagine how surprised I was when I saw this crash for the first time. Whaaat? Caaache? But how? :)
As for me, removing fatalError
seems to be enough to eliminate negative impact. It's just an overkill here.
from networking.
Related Issues (20)
- Support for JSONDecoder HOT 8
- Token Refresh HOT 6
- Json parameters support on DELETE HOT 3
- Progress Multipart Upload HOT 2
- Support PATCH method HOT 1
- Network Activity Indicator not showing
- Change the base URL for a single request? HOT 1
- Installation Support Swift Package Manager HOT 2
- Caching folder should not be documents folder HOT 1
- fakeGET crashes under XCode 10.0 and Swift 4.2
- Expose handleJSONRequest(..)? HOT 3
- Problem with changes on UI Threath HOT 3
- `cacheOrPurgeData` causing production crashes HOT 3
- Can't use Swift 5 Result enum due to interference with custom one. Need to rename? HOT 3
- I didn't found way to get my body data as json parsing is failing HOT 7
- No waitsForConnectivity support for background request. HOT 1
- Using Networking lib to make a post in the background? HOT 3
- Question - Request caching HOT 2
- Troubleshooting Logging Errors in Xcode
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from networking.