Commit Graph
394 Commits
Author SHA1 Message Date
jvoisinandGitHub 828c1b2674 Use modern Spring mapping annotations
Java is verbose enough, no need to make it worse.
2019-06-06 18:52:11 +00:00
jvoisin 3492fc0c05 Fix a crash when transcoding a track without album 2019-06-06 02:19:08 +02:00
jvoisin b1eb0536b8 Fix a crash when transcoding a track without artist 2019-06-06 02:17:10 +02:00
jvoisin e283ac46a8 Fix downloads with embedded covers
Previously, this would always fail with
embedded covers.
2019-06-06 02:14:10 +02:00
jvoisin b697dc5bfc An other batch of LGTM-found issues 2019-06-05 23:40:10 +02:00
Schuyler Eldridge 89a4517f64 Show cover art not displaying on chromecast
Signed-off-by: Schuyler Eldridge <schuyler.eldridge@gmail.com>
2019-06-05 00:01:21 -04:00
jvoisinandGitHub 4f59c72bbe Replace wz_tooltip.js with jquery-ui 2019-06-04 22:27:47 +00:00
jvoisinandGitHub 6aa96911de Replace FancyZoom with jquery.fancyzoom 2019-06-04 22:13:07 +00:00
jvoisinandGitHub e3a4fe4603 Balance some synchronized 2019-06-04 21:08:57 +00:00
jvoisinandGitHub 4263d59940 Fix a possible stacktrace on RandomPlayQueue 2019-06-04 21:06:08 +00:00
jvoisinandGitHub 24632317b2 Declare the usage of HTML5 in the doctype
This should marginally increase the performances
and jquery (no more quirks mode), as well
as making web browsers happier.
2019-06-04 21:04:59 +00:00
jvoisin fe73034f10 Strip the <p>…</p> markup in translation
This was done via `find airsonic-main/src/main/resources/ -name
'ResourceBundle_*.properties' -print0 | xargs -0 sed -i
's/\(helppopup\..*\.text\)\s*=\s*<p>\(.*\)<\/p>/\1=\2/'`, courtesy of fxthomas.
2019-05-30 00:49:54 +02:00
jvoisin 01dd070efa Fix indentation 2019-05-30 00:49:35 +02:00
jvoisin 195a0879c3 Replace wz_tooltip.js with jquery-ui
Since we're already using jquery-ui for other stuff, it makes sense to use it
for tooltips too, instead of using an old-school-written-in-2009 1300 lines
  steaming pile of javascript named wz_tooltip.js
2019-05-29 23:39:25 +02:00
jvoisin f4fa75f808 Fix compat with the latest jquery
Stolen^w borrowed from https://github.com/keegnotrub/jquery.fancyzoom/issues/3#issue-388966769
2019-05-28 01:13:02 +02:00
jvoisin c5eb7aa37c Fix a typo 2019-05-28 01:08:27 +02:00
jvoisin 9a43b4f360 Group the .attr calls 2019-05-28 00:57:26 +02:00
jvoisin 15a1e5e7d6 Replace FancyZoom with jquery.fancyzoom
FancyZoom isn't a free-as-in-freedom software,
and is mostly a big pile of old-school javascript.
This commit replace it with a small jquery-powered
script under MIT license.
2019-05-28 00:56:46 +02:00
jo 131b7ada1d Update translation and fix char encoding 2019-05-23 21:12:42 +02:00
tesshucom 1c4a70af99 Fixed null check with StringUtils/ObjectUtils. 2019-05-24 03:57:57 +09:00
Andrew DeMaria ce2a56e6f1 Merge remote-tracking branch 'origin/pr/1033' 2019-05-23 10:45:07 -06:00
Peter Marheine 3efa2d3e2c Remove the option to never set Content-Length
This workaround is obsolete with correct handling of ranges when
requested.
2019-05-23 10:54:06 +10:00
Peter Marheine 9be2a8892f More correctly handle stream ranges
When transcoding, always use chunked transfers and report that ranges
are not supported. When not transcoding, support returning ranges but
only if requested.
2019-05-23 10:51:37 +10:00
Peter Marheine 3e97186043 Omit unnecessary null check on ranges 2019-05-23 10:50:20 +10:00
Peter Marheine 47eefc1369 Refactor output streaming for readability
The core loop was put together confusingly; this encapsulates the
process of constructing an OutputStream and reformats some.
2019-05-23 10:48:09 +10:00
Peter Marheine cae8f8b4e5 Reformat stream controller some
Mostly just whitespace changes, shortening lines for readability. Adds a
few comments that outline what's supposed to happen.
2019-05-23 10:43:51 +10:00
Peter Marheine d42af4575f Add some @Nullable annotations
Making it easier to tell where API contracts allow nulls, where it's
otherwise unclear without reading the implementation.
2019-05-23 10:37:35 +10:00
Peter Marheine 5077e0d5e1 Short circuit RangeOutputStream for open ranges
There's no reason to wrap a stream in an output that will do nothing, as
when the requested range is 0-; eg the entire stream.
2019-05-23 10:34:57 +10:00
Andrew DeMaria 2643545b39 Rename utilsjs 2019-05-21 11:00:27 -06:00
Andrew DeMaria 493a53b418 Merge remote-tracking branch 'origin/pr/930' 2019-05-20 15:30:15 -06:00
Andrew DeMaria bec5cec4a2 Merge remote-tracking branch 'origin/pr/930' 2019-05-20 15:05:18 -06:00
jvoisin 1090348b0e Use the version number in jquery-ui's css path 2019-05-20 22:57:21 +02:00
Andrew DeMaria 0a1164966d Merge remote-tracking branch 'origin/pr/1032' 2019-05-20 14:52:32 -06:00
Andrew DeMaria cdd307d86f Merge remote-tracking branch 'origin/pr/1042' 2019-05-20 14:50:56 -06:00
jvoisinandGitHub ce7671bf5e Add the required keyword to some forms
This should improve a bit the accessibility.
2019-05-17 20:29:16 +00:00
jvoisinandGitHub 6d294a770b Replace the usage of prototypejs with jquery in changeCoverArt.jsp
This change was successfully tested on Chromium and Firefox.
2019-05-17 20:23:40 +00:00
jvoisinandGitHub bf5f8cb713 Factorize scripts-2.0.js into utils.js
No need to include two different and super-small javascript "utils" files.
2019-05-17 20:13:12 +00:00
jvoisinandGitHub f3b2c2ea7a Add a missing taglib to homePager.jsp
The missing taglib is confusing coverity a bit,
and I think that it's a good practise to add it
anyway.
2019-05-17 17:13:01 +00:00
jvoisinandGitHub 3cfe2b31a1 Move some cast-related inline js to an external file
There is no point in having such a massive
blob of javascript inline in the page.
2019-05-17 17:09:45 +00:00
jvoisinandGitHub ab03526620 Fix two NULL-deref 2019-05-01 08:52:09 +00:00
François-Xavier Thomas f57ad3f27b Fix typo in anonymous user name (#663) 2019-04-29 23:30:02 +02:00
Andrew DeMaria ac0a722a10 Merge remote-tracking branch 'airsonic/pr/990' 2019-04-28 22:59:23 -04:00
Andrew DeMaria b51cdc1c5c Merge remote-tracking branch 'airsonic/pr/1028' 2019-04-28 22:58:29 -04:00
François-Xavier Thomas 8a90d9f77b Add system properties for persisting the 'remember me' key
This adds the 'airsonic.rememberMeKey' system property (can be set from
command-line with `-Dairsonic.rememberMeKey=<value>`) as well as a
'RememberMeKey' setting in airsonic.properties, so that the key used for
generating 'remember me' tokens can be persisted across server restarts.

It also adds a default, insecure key in case we are running in
development mode with the 'airsonic.development' property set.
2019-04-28 16:51:46 +02:00
jvoisin 3ee6fefe11 Bump jQuery/jquery-ui to the latest versions
I bumped it first to 2.X with jQuery migrate, played around but didn't manage
to trigger any warning.  So I bumped it again to the latest available version,
jQuery 3.4.0, which isn't triggering useful warnings either.
2019-04-28 16:26:36 +02:00
jvoisinandGitHub b663a2fb90 Fix a stored XSS 2019-04-28 08:49:25 +00:00
jvoisinandGitHub 348c698e35 Remove the /db page
This page wasn't linked anywhere, and was
allowing an administrator to issue arbitrary sql
comments, and was vulnerable to reflected XSS.

We should get rid of it. If you really want to issue
SQL commands, just ssh to your instance and do it from here.
2019-04-28 08:48:41 +00:00
jvoisinandGitHub d3970a5c62 Fix various minor issues found by LGTM
- Unnecessary boxing
- Integer overflow
- Path traversal via zip
- Dangerous synchronization pattern
2019-04-28 08:37:47 +00:00
Andrew DeMaria a911ebab80 Merge remote-tracking branch 'origin/pr/1027' 2019-04-27 20:27:37 -06:00
Andrew DeMaria 2162250101 Revert "Minor frameset-related factorization"
This reverts commit e69287cfe6.

Signed-off-by: Andrew DeMaria <lostonamountain@gmail.com>
2019-04-27 14:00:52 -06:00