Giter Club home page Giter Club logo

Comments (5)

lesserwhirls avatar lesserwhirls commented on July 18, 2024 1

Both of these issues (encoding and links to docs) should be fixed in 4.6.x and 5.0 now. Thank you again for your report!

from thredds.

meliezer avatar meliezer commented on July 18, 2024

It also happens for spaces in the path. For some reason the GET includes %2520 instead of %20 for spaces.

from thredds.

lesserwhirls avatar lesserwhirls commented on July 18, 2024

There is some javascript used in the OPeNDAP html data access page that makes a web request when you click "Get Binary" or "Get ASCII", and that code makes sure to encode the URL before making those requests. It currently encodes the entire URL, but it looks like it should only encode the query portion of the URL. What happens is the percent sign in the encoded space gets encoded a second time (to %25) on submission. It should be a relatively simple fix though. Thanks for the report!

from thredds.

Alexander-Barth avatar Alexander-Barth commented on July 18, 2024

Thanks for your feedback! I think that this is indeed the issue.

The javascript code in the HTML page:

function ascii_button() {
    var url = new String(document.forms[0].url.value);

    var url_parts = url.split("?");
    /* handle case where constraint is null. */
    if (url_parts[1] != null) {
        var ascii_url = url_parts[0] + ".ascii?" + url_parts[1];
    }
    else {
        var ascii_url = url_parts[0] + ".ascii?";
    }

    window.open(encodeURI(ascii_url), "ASCII_Data");
}

I guess that only url_parts[1] should be encoded (as you mentioned).
In the download form, Action, Data URL, Global Attributes, Variables are (currently) broken links to like links these:

https://www.opendap.org/online_help_files/opendap_form_help.html#data_url

I was wondering if you were aware of this.

Thanks again for your help!

from thredds.

Alexander-Barth avatar Alexander-Barth commented on July 18, 2024

This is great! Do you have already an idea when will be a release with this fix? Thanks a lot!

from thredds.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.