Enhancement #894

URL test does not work

Added by Mérour Xavier almost 3 years ago. Updated over 2 years ago.

Status:ClosedStart date:01/19/2015
Priority:NormalDue date:
Assignee:-% Done:

100%

Category:SHOP
Target version:4.3.1
Sponsor:ASIT-VD (CH) Ergonomic impact:

-

Functional impact:

-


Description

In resource management, in diffusion options, the user can choose product storage = URL under "Download" (See attached image).
Then he's asked for the file URL and, optionally , for login/password. Then the user can test the URL Access.

It seems that this feature only test the domain name :

- if I test www.easysdi.org, it returns "The resource is accessible thanks to the information provided"
- if I test "easysdi.org/fffff" (which is HTTP error 404), the test returns a "The resource is accessible thanks to the information provided". It should return an error message ("The resource is accessible thanks to the information provided").

History

#1 Updated by Mérour Xavier almost 3 years ago

  • Tracker changed from Defect to Enhancement
  • Assignee set to Blatti Yves

After some discussion, the current test is indeed only on the domain name.

This should be enhanced with a complete URL test (on headers returned).

#2 Updated by Mérour Xavier almost 3 years ago

  • Status changed from New to Request For Comments
  • Assignee changed from Blatti Yves to Technical Committee

To be discussed first in TC

#3 Updated by Van Hoecke Hélène almost 3 years ago

  • Assignee changed from Technical Committee to Magoni Bruno

TC validates that the test must be on complete URL and should be implemented by Head Request.
TC proposes that this functionnality was located in the CORE to be factorisable.

#4 Updated by Magoni Bruno almost 3 years ago

  • Status changed from Request For Comments to Accepted
  • Assignee deleted (Magoni Bruno)
  • Target version set to Unplanned

#5 Updated by Mérour Xavier almost 3 years ago

  • Assignee set to Blatti Yves

#6 Updated by Mérour Xavier almost 3 years ago

Please complete description with code HTTP handling (+ localization)

#7 Updated by Blatti Yves almost 3 years ago

TC has accepted to relocate this method in CORE to be transversal.

I would suggest that this URL check helper would have the following features:
  • Handle HTTP, HTTPS, FTP, (FTPS if possible with php curl)
  • Optional authentication (user/password)
  • Follow redirections
  • Do not load full files (they can be really heavy), instead use HTTP HEAD (CURLOPT_NOBODY), or load only partial file with GET (in case of HTTP). What about FTP ?
  • Return an object with:
    • Success (true/false) : true only if file is reachable
    • Code (200, 404, 403, ...)
    • Message ('Found','Not found','Forbidden',...)
  • The user of the library can use only the status, or can give a more detailed message if needed (code + message)
  • I would suggest that this library does not carry any translations

#8 Updated by Mérour Xavier almost 3 years ago

  • Status changed from Accepted to Affected
  • Assignee changed from Blatti Yves to Magoni Bruno

#9 Updated by Magoni Bruno almost 3 years ago

  • Sponsor ASIT-VD (CH) added

#10 Updated by Magoni Bruno over 2 years ago

  • Target version changed from Unplanned to 174

#11 Updated by Magoni Bruno over 2 years ago

  • Target version changed from 174 to 4.3.1

#12 Updated by Magoni Bruno over 2 years ago

  • Assignee changed from Magoni Bruno to Villemagne Jérôme

#13 Updated by Villemagne Jérôme over 2 years ago

  • Status changed from Affected to Resolved
  • Assignee changed from Villemagne Jérôme to Mérour Xavier
  • % Done changed from 0 to 100

#14 Updated by Van Hoecke Hélène over 2 years ago

  • Status changed from Resolved to Closed

#15 Updated by Van Hoecke Hélène over 2 years ago

  • Assignee deleted (Mérour Xavier)
  • Functional impact set to -
  • Ergonomic impact set to -

Also available in: Atom PDF