Resolving relative URIs not working properly

Bug #866819 reported by Gabriel Petrovay
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Zorba
New
Low
Matthias Brantner

Bug Description

URI resolving implemented in the URI class is not working properly.
This functionality is exposed in the API in the StaticContext::resolve() functions.

The corresponding test (test/api/test_static_context) is failing or this reason. See the examples in:
zorba\test\api\test_static_context.cpp

Moreover, the fn:resolve-uri implementation (zorba\src\runtime\any_uri\any_uri_impl.cpp) should be implemented in the URI class (zorba\src\zorbatypes\URI.cpp). Only the exceptions should be handled in the fn:resolve-uri implementation.

Revision history for this message
Gabriel Petrovay (gabipetrovay) wrote :

the expected failure is in: zorba\test\api\CMakeLists.txt

Revision history for this message
Gabriel Petrovay (gabipetrovay) wrote :

Check out:
fn:resolve-uri("/..", "http://example.com/")
or
fn:resolve-uri("/../dir1/dir2/../../../dir3/../dir/file", "http://example.com/")

added these tests to test/unit/uri which is now also marked as fialure in zorba\test\unit\CMakeLists.txt because of this same bug.

Revision history for this message
Matthias Brantner (matthias-brantner) wrote :

decreasing the priority of this. It's a corner case and not doable before the 2.0 release.

Chris Hillery (ceejatec)
tags: removed: core-runtime
tags: removed: v2.1
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.