It appears that the SSL Version config setting isn't actually used anywhere, or even properly stored. It also is a required but empty field on the settings form, which is really bad UX. Seems like we should just kill it. Patch to follow.

CommentFileSizeAuthor
#3 stage_file_proxy-2605102-3.patch2.8 KBDane Powell
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dane Powell created an issue. See original summary.

Dane Powell’s picture

Issue summary: View changes
Dane Powell’s picture

Status: Active » Needs review
FileSize
2.8 KB
markdorison’s picture

Status: Needs review » Reviewed & tested by the community

Patch works as described.

  • greggles committed 500578d on 8.x-1.x authored by Dane Powell
    Issue #2605102 by Dane Powell | markdorison: Stale SSL version setting
    
greggles’s picture

Status: Reviewed & tested by the community » Fixed

Thanks for the code and review. Now committed.

As I said on #2671094: Double encoding filename causes assets to not render: BTW, I'd be happy to accept a co-maintainer (especially who cares about 8.x) if someone wants to apply :)

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.