Bond-009
07326c1d9b
Merge branch 'master' into fix-auth-response-codes
2020-04-22 13:16:08 +02:00
dkanada
d62bd7fecd
fix spelling error
2020-04-19 11:46:22 +09:00
Vasily
95dc99fdbd
Update Emby.Server.Implementations/HttpServer/HttpResultFactory.cs
...
Co-Authored-By: Bond-009 <bond.009@outlook.com>
2020-04-16 01:03:29 +03:00
Vasily
9df49cc796
Make Last-Modified and If-Modified-Since headers follow the spec
2020-04-14 01:52:43 +03:00
Mark Monteiro
8b4b4b4127
Do not return the exception message to the client for AuthenticationExceptions
2020-04-13 16:10:55 -04:00
Mark Monteiro
a8c3951c17
Only show developer exception page for 500 server exceptions
...
Other response codes should be returned as normal
2020-04-13 15:26:49 -04:00
Mark Monteiro
53380689ad
Return correct status codes for authentication and authorization errors
...
- Use AuthenticatonException to return 401
- Use SecurityException to return 403
- Update existing throws to throw the correct exception for the circumstance
2020-04-13 13:17:46 -04:00
Mark Monteiro
6d35dd6b32
Clean up SecurityException
...
- Remove unused SecurityExceptionType
- Add missing constructor for InnerException
- Add missing documentation
2020-04-13 13:13:48 -04:00
Vasily
3bdb5e80a5
More consise error messages
2020-04-13 00:57:11 +03:00
Vasily
058c35e739
Fix log highlithing
2020-04-12 23:40:34 +03:00
Vasily
30f4392872
Fix condition flipped by https://github.com/jellyfin/jellyfin/pull/2635
2020-04-12 23:35:41 +03:00
Vasily
61d9c9df5b
Addressing review feedback
2020-04-12 23:26:45 +03:00
Vasily
71d8e66d9f
Add logging of URL being processed when logging an error
...
This might help diagnosing stuff like "Operation was cancelled"
2020-04-06 14:42:41 +03:00
Mark Monteiro
de634203d8
Put Boolean operators at beginning of lines instead of the end
2020-04-02 14:31:56 -04:00
Mark Monteiro
eadad73117
Merge branch 'master' into use-development-exception-page
2020-04-02 13:51:53 -04:00
Mark Monteiro
123bfbcf19
Merge remote-tracking branch 'upstream/master' into use-development-exception-page
2020-04-01 23:09:55 +02:00
Mark Monteiro
5b1091db27
Merge remote-tracking branch 'upstream/master' into do-not-instantiate-services-at-startup
2020-04-01 23:05:36 +02:00
Bond-009
f31efce52d
Merge branch 'master' into usings
2020-04-01 22:42:43 +02:00
Bond_009
e9d1eabd53
Remove unused usings
2020-03-24 16:12:06 +01:00
Mark Monteiro
72c822513f
Create ServiceController via DI instead of manual construction
2020-03-21 23:18:41 +01:00
Mark Monteiro
dc140429fe
Do not create unnecessary instances of IService at server startup
2020-03-21 23:17:30 +01:00
Mark Monteiro
842ec04828
Do not handle exceptions manually when in development mode
2020-03-21 21:06:01 +01:00
Mark Monteiro
92628c4033
Clean up HTTP listener exception handling
2020-03-21 21:04:16 +01:00
Mark Monteiro
e2685c51ec
Merge remote-tracking branch 'upstream/master' into support-running-without-web-content
2020-03-15 18:07:02 +01:00
Mark Monteiro
6fbdf0d6a2
Construct ApplicationHost with DI framework instead of manually
2020-03-15 15:28:29 +01:00
Bond-009
94fe9b8f6d
Merge branch 'master' into warn17
2020-03-11 20:26:43 +01:00
Mark Monteiro
3f4b9e9a81
Add new 'nowebcontent' configuration flag
2020-02-28 20:40:45 +01:00
Mark Monteiro
3043b7323b
Use constants for settings keys
2020-02-28 17:57:38 +01:00
Bond_009
07cc4be6a7
Fix some warnings
...
* Add analyzers to MediaBrowser.XbmcMetadata
* Enable TreatWarningsAsErrors for MediaBrowser.XbmcMetadata
* Add analyzers to MediaBrowser.WebDashboard
* Enable TreatWarningsAsErrors for MediaBrowser.WebDashboard
* Disable SA1600 in favor of CS1591
2020-02-23 12:11:43 +01:00
Bond_009
7060934792
Fix some warnings in Emby.Server.Implementations
2020-02-06 15:21:38 +01:00
Bond_009
c751ba9f70
Fix build
2020-01-22 22:27:03 +01:00
Bond-009
c8409d2ea1
Remove FileSystem.GetStream
2020-01-22 22:20:35 +01:00
Bond_009
2ef4ffd698
More warnings (removed)
2019-12-11 00:13:57 +01:00
Bond-009
affb58ef9e
Apply suggestions from code review
...
Co-Authored-By: dkanada <dkanada@users.noreply.github.com>
2019-12-10 16:22:03 +01:00
Bond_009
42ffddc269
Fix more warnings
2019-11-27 16:29:56 +01:00
Vasily
5263aaa026
Merge pull request #1503 from cvium/webapi_v2
...
Move StartupWizard to ASP.NET Web Api
2019-11-27 11:50:06 +03:00
Claus Vium
2af5922af0
Fix review comments
2019-11-24 18:25:43 +01:00
Claus Vium
27e3cf1558
Move appbuilder and service collection to Jellyfin.Server
2019-11-24 15:27:58 +01:00
Claus Vium
3f651de24c
Add authentication and remove versioning
2019-11-23 16:31:02 +01:00
Bond-009
b477b3874e
Fix some warnings
2019-11-22 16:15:31 +01:00
Bond-009
b14d6d0417
Fix GetPathValue function
2019-11-13 16:32:33 +01:00
Vasily
6b6fede2e0
Address review comments
2019-10-16 16:13:59 +03:00
Vasily
82f8345aa5
Log to debug all HTTP 500 response urls
2019-10-14 13:22:53 +03:00
Vasily
1176749f14
Merge pull request #1863 from joshuaboniface/fix-baseurl-issues
...
Fix inconsistent BaseUrl behavior
2019-10-09 19:24:41 +03:00
Joshua Boniface
b96079fee6
Make NormalizeUrlPath static
2019-10-09 09:22:55 -04:00
Vasily
03450f383f
Fix template for logging
2019-10-09 14:05:32 +03:00
Vasily
8109c7eb30
Always log at least error message when error happens during request processing
2019-10-09 14:00:22 +03:00
Vasily
33b69a7099
Set response length and mime type correctly when reporting an error
2019-10-09 13:54:05 +03:00
Joshua Boniface
2a79ae0a6e
Normalize baseUrl behaviour
...
Fully normalizes the baseUrl behaviour to better match how this sort of
feature works in other programs.
1. The baseUrl is always appended to paths, even the built-in `/emby`
and `/mediabrowser` paths.
2. The baseUrl is set statically at class instance creation, to ensure
it persists through changes until the next restart.
3. Configuration is normalized using a function when set, to ensure it's
in a standard `/mypath` format with leading `/`.
4. Cleans up the conditionals around default redirects. For sanity after
changing the URL, it will match *any* path that doesn't match the
current baseUrl and redirect it back to the main page (with baseUrl).
5. Adds a second method, NormalizeUrlPath, to avoid lots of `+ "/" +`
string manipulations which are unclean - we should always have a leading
slash.
6. Sets the default baseUrl to an empty string to avoid unexpected
behaviour, though this would be worked-around automatically.
7. Adds some debug logs whenever a URL is normalized, to help track down
issues with this code (if any arise).
2019-10-08 18:06:03 -04:00
Vasily
fc99f1f563
Merge pull request #1785 from dkanada/compat
...
Add mediabrowser route back for now
2019-09-25 14:18:21 +03:00