http --> https correction

Unidata / thredds

THREDDS Data Server v4.6

https://www.unidata.ucar.edu/software/tds/v4.6/index.html

265 stars 179 forks source link

http --> https correction #1322

Closed oxelson closed 4 years ago

oxelson commented 4 years ago

@lesserwhirls http --> https protocol correction for unidata website. Ignored/skipped references containing or pertaining to:

xmlns
dtd
schema
namespace
obvious code invocations

lesserwhirls commented 4 years ago

Thanks @oxelson! In addition to the extra s, we have a new test failure:

thredds.client.catalog.TestResolve1 > testResolve FAILED
    java.lang.AssertionError: https://www.unidata.ucar.edu/
        at thredds.client.catalog.TestResolve1.testResolve(TestResolve1.java:70)

which is due to this:

docUrl.equals("http://www.unidata.ucar.edu/")

I'm thinking there might be more like that once we run our full tests on Jenkins.

oxelson commented 4 years ago

@lesserwhirls, fixed the assert docUrl test to use https. Passing travis now. Understood about the possibility of more failures on jenkins.

lesserwhirls commented 4 years ago

Thanks @oxelson! I'll merge this is and start a run on Jenkins to see how things go.

lesserwhirls commented 4 years ago

Actually, going to run current master on jenkins first, to get a baseline.