Updates: Mandatory Fun
So I mentioned in my previous post that, with the powers granted to me by Mojolicious, I have migrated from the venerable Apache server with all my server-side fancy stuff being CGI to Docker containers. Awesome! This morning my buddy n4vn33t shared with me an interesting blog post; I'll let the post speak for itself but my takeaway was essentially, sure that upstream Docker image I'm using ought to be patched and ready to rock but... What if it's not? It would help to add
RUN apt-get -y upgrade to my
Dockerfile to make sure I've got the latest and greatest stuff. And so I did. And then I ran my container locally to give it a quick test and now my remaining CGI scripts are returning 404s. If I run the app locally with
morbo www-swagg.pl it “just works” so I must've borked my Docker container!
To troubleshoot, I comment out the
apt-get -y upgrade stuff and rebuild. That should yield the same result as prior to the change right? Well, no it didn't. At this point I'm lost so I begin removing all traces of previous images, re-pulling and re-building things. Still got 404s for my beloved guestbook which I think goes without saying, is completely unacceptable! Then I thought about this line in my
RUN cpanm RJBS/Getopt-Long-Descriptive-0.105.tar.gz
Why run that instead of
cpanm Getopt::Long::Descriptive? You see, this module isn't one that I'm using in my scripts but rather it's a dependency; a module I'm using is using this module. One day (prior to this) my Docker container refused to build and I narrowed the problem down to v0.106 of Getopt::Long::Descriptive building and, as it's not a module I'm much concerned with (I just need it so the module I do need will build) I enter the command as you see above to force installation of the prior version that I know will build just fine. So I start by checking out the changelogs for my modules that I'm using in my scripts. Thankfully it didn't take me long to see this in the changelog for Mojolicious (first module I checked 🙃):
– This release contains fixes for security issues, everybody should upgrade!
– Disabled format detection by default to fix vulnerabilities in many Mojolicious applications. That means some of
your routes that previously matched “/foo” and “/foo.json”, will only match “/foo” after upgrading. From now on you
will have to explicitly declare the formats your routes are allowed to handle.
$r->get('/foo' => [format => ['html', 'json']])–>to('bar#yada', format => undef);
And if you are certain that your application is not vulnerable, you also have the option to re-enable format
detection for a route and all its nested routes. Due to the high risk of vulnerabilities, this feature is going to
be removed again in a future release however.
my $active = $r->any([format => 1]);
Ahh that's right... I'm doing this:
plugin 'Config'; # CGI scripts plugin CGI => ['/cgi-bin/guest' => './cgi-bin/guest_mm.cgi']; plugin CGI => ['/cgi-bin/whoami' => './cgi-bin/whoami.cgi' ];
.cgi file extension wasn't technically necessary but I want that to still work because:
- I already have hyperlinks all over the place that use the extension
- It's a CGI script and I want it to “look” like that... Petty I know
So here's my v9.11+ compliant way of using the extension:
plugin CGI => ['/cgi-bin/guest.cgi' => './cgi-bin/guest_mm.cgi']; plugin CGI => ['/cgi-bin/whoami.cgi' => './cgi-bin/whoami.cgi' ];
Excellent, we're back in business! Fighting issues like this can sometimes feel like a “waste” of an afternoon because at the end of the day... My site hasn't really changed. I gained no new fun buttons or GIFs but I can sleep easy tonight knowing that my site is just a bit more “hardened” against script kiddies who never cease to make our lives just a little bit more complicated. Let's that tag this puppy, push it to my cloud provider and call it a day:
# After we've run: docker build -t www-swagg . docker tag www-swagg gcr.io/www-swagg/www-swagg docker push gcr.io/www-swagg/www-swagg # Bunch of output follows...
And now we're safe. Until tomorrow when the next round of vulnerabilities gets discovered anyways 🤦♂️