-
Notifications
You must be signed in to change notification settings - Fork 29
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
add convenience helper function for constructing file:///
URIs
#60
base: master
Are you sure you want to change the base?
Conversation
Codecov Report
@@ Coverage Diff @@
## master #60 +/- ##
==========================================
- Coverage 86.78% 85.61% -1.17%
==========================================
Files 5 5
Lines 280 292 +12
==========================================
+ Hits 243 250 +7
- Misses 37 42 +5
Continue to review full report at Codecov.
|
There is a similar effort underway at rofinn/FilePaths.jl#22. I think that would almost be cleaner, the syntax then might be |
Similar, but this one is a bit more powerful: |
Are query strings and fragments allowed in the file URI scheme? This seems to suggest not? |
That RFC appears to have been very recently proposed. The current internet standard document for URI objects (https://tools.ietf.org/html/rfc3986) doesn't specify dependence of the parse syntax being dependent on the scheme being some specific string. |
Well, we are following some guidelines on how to convert a path to an URI. RFC3986 doesn't say anything about that, so aren't we implicitly following either rfc1738 or the new rfc8089 (which seems quite reasonable)? Neither of which seems to support fragments/queries. |
Neither of those are standards though. RFC3986 is a standard, and includes explicit guidance on the meaning and parsing of a relative URI, how to combine it with a base URI, mentions some guidance on its applicability to the |
I think an alternative to this PR might be the combination of rofinn/FilePathsBase.jl#1 and rofinn/FilePaths.jl#28, or some extension of those. |
Bump. Updated this with code to handle the drive-spec on Win32 (where the translation is defined, according to rfc8089) |
From PR davidanthoff/Electron.jl@0108a25