? ? Success

User tests: Successful: Unsuccessful:

avatar zero-24
zero-24
18 Apr 2017

Pull Request for Issue #15381

Summary of Changes

Add useragent for update calls

Testing Instructions

test 1
  • install a outdated extension
  • apply this patch
  • run the update
  • confirm everything works
test 2
  • install a outdated joomla version
  • patch the file manually
  • update to the last joomla version

Expected result

Update works with the useragent checks

Actual result

Update works but have no useragent

Documentation Changes Required

None

90ba824 18 Apr 2017 avatar zero-24 cs
avatar zero-24 zero-24 - open - 18 Apr 2017
avatar zero-24 zero-24 - change - 18 Apr 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 18 Apr 2017
Category Libraries
avatar zero-24 zero-24 - change - 18 Apr 2017
The description was changed
avatar zero-24 zero-24 - edited - 18 Apr 2017
avatar zero-24 zero-24 - change - 18 Apr 2017
The description was changed
avatar zero-24 zero-24 - edited - 18 Apr 2017
avatar zero-24 zero-24 - change - 18 Apr 2017
Labels Added: ?
avatar Dade88
Dade88 - comment - 18 Apr 2017

As from original #15381 I've tested by the following tests:

  1. Re-enable mod_security directives on my updates sub-domain
  2. Apply changes to a Joomla installation, while another stay 'as is'
  3. Install a self-made extension pointing to my update server

Result for the installation with changes receives the ok from the server (Code 200) with a defined User Agent (Mozilla/5.0 Joomla!/3.6.5 Joomla) . From Apache logs:

213.251.182.102 dev.tilellit.pro - [19/Apr/2017:01:14:02 +0200] "GET /joomla/adminer.xml HTTP/1.1" 200 902 "-" "Mozilla/5.0 Joomla!/3.6.5 Joomla"

The Joomla install without modification receives an error 403 and no user agent at all. Again Apache log:

213.251.182.102 dev.tilellit.pro - [19/Apr/2017:01:13:44 +0200] "GET /joomla/adminer.xml HTTP/1.1" 403 220 "-" "-"

Good Work!

avatar zero-24 zero-24 - change - 14 Jun 2017
Milestone Added:
avatar rdeutz rdeutz - change - 14 Jun 2017
Milestone Added:
avatar rdeutz rdeutz - change - 14 Jun 2017
Milestone Removed:
avatar roland-d roland-d - test_item - 10 Jul 2017 - Tested successfully
avatar roland-d
roland-d - comment - 10 Jul 2017

I have tested this item successfully on f9f3dd5

Before the patch there is no user agent in the request:

"GET /rep/extension.xml HTTP/1.1" 200 720 "-" "-"

After the request the user agent is present:

"GET /rep/extension.xml HTTP/1.1" 200 720 "-" "Mozilla/5.0 Joomla!/3.7.4-dev Joomla"

Works great :)


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/15385.

avatar Quy
Quy - comment - 10 Jul 2017
avatar franz-wohlkoenig franz-wohlkoenig - change - 10 Jul 2017
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 10 Jul 2017

RTC as stated above.

avatar rdeutz rdeutz - change - 11 Jul 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-07-11 14:13:34
Closed_By rdeutz
Labels Added: ?
avatar rdeutz rdeutz - close - 11 Jul 2017
avatar rdeutz rdeutz - merge - 11 Jul 2017

Add a Comment

Login with GitHub to post a comment