PR-staging

Pending

User tests: Successful: Unsuccessful:

avatar zero-24
zero-24
26 Jul 2017

Pull Request for Issue #17247

Summary of Changes

Now we are stripping and checking the ports too.

Testing Instructions

Database host in following cases are considered as remote server (wrong detection)

  1. 127.0.0.1:port (Eg, in case database server not run on default port)
  2. a.b.c.d (same ip address with web server)
  3. localhost:port (cannot connect)

Expected result

Works

Actual result

does detect as remote database

Documentation Changes Required

avatar joomla-cms-bot joomla-cms-bot - change - 26 Jul 2017
Category Installation
avatar zero-24 zero-24 - open - 26 Jul 2017
avatar zero-24 zero-24 - change - 26 Jul 2017
Status New Pending
avatar Song2017
Song2017 - comment - 28 Jul 2017

Hi,
When I deploy to azure web app with database password 6#vWHD_$ and host like 127.0.0.1:49858, it fails.
After delete verify file, it hangs on steps 4 creating configuration File during Installing, then Got this

The most recent request was denied because it contained an invalid security token. 
Please refresh the page and try again.

Is there a way to find log or some solution?
Thanks.

avatar zero-24 zero-24 - change - 28 Jul 2017
Labels Added: PR-staging
avatar zero-24
zero-24 - comment - 28 Jul 2017

@Song2017 do you have ftp access on that azure thing? If yes you need to appy this patch + #17248 together. Please report back if that fixes the issue or not. Thanks

avatar Song2017
Song2017 - comment - 31 Jul 2017

@zero-24 It fixed the issue.
With applying fixinstaller.zip, Joomla version 3.8:0-dev has been installed on azure successfully.
BTW, it would be grateful you tell us when a fixed version will be posted.
Thanks

avatar franz-wohlkoenig franz-wohlkoenig - alter_testresult - 31 Jul 2017 - Song2017: Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 31 Jul 2017

@Song2017 i altered your Test as successfully.

For Informations you can find Timeline of 3.8 when this Version is planned to be shipped or order a Newsletter.

avatar Song2017
Song2017 - comment - 31 Jul 2017
avatar zero-24 zero-24 - change - 14 Aug 2017
Labels Added: ?
avatar franz-wohlkoenig franz-wohlkoenig - change - 24 Aug 2017
Title
Check for ports and make sure we are haveing local posts
Check for ports and make sure we are having local posts
avatar joomla-cms-bot joomla-cms-bot - edited - 24 Aug 2017
avatar nibra nibra - change - 15 Sep 2017
Priority Medium Urgent
avatar fastslack
fastslack - comment - 17 Sep 2017

@test

I'm testing on linux and mysql on port 3306. Latest Joomla! 3.8.0 package for test/

I tried different database hosts:

  • 127.0.0.1 Works as expected
  • 127.0.0.1:3306 Works as expected
  • localhost:3307 <- Seems to be an issue here. This pass checks and continue with installation
  • localhost Works as expected
  • localhost:3306 I got the message
You are trying to use a database host which is not on your local server. For security reasons, you need to verify the ownership of your web hosting account. Please read the documentation for more information.

After remove _JoomlaKeyxaZVqes2lzX4xcsWsi.txt file works as expected

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 17 Sep 2017

@fastslack if i understand correct you test unsuccessfully.

Please mark Test:

  • Please open Issue Tracker
  • Login with your github-Account
  • Click on blue "Test this"-Button above Authors-Picture
  • mark your Test as unsuccessfully
  • hit "submit test result"
avatar zero-24
zero-24 - comment - 19 Sep 2017

localhost:3307 <- Seems to be an issue here. This pass checks and continue with installation

Can you explain your reasoning? I don't understand what did happen or what do you mean by " This pass checks and continue with installation"

avatar zero-24
zero-24 - comment - 19 Sep 2017

localhost:3306 I got the message

Can you please double check there are no spaces etc in that string? As localhost and 127.0.0.1 has the same code ;)

avatar franz-wohlkoenig franz-wohlkoenig - change - 1 Oct 2017
Priority Urgent Medium
avatar zero-24
zero-24 - comment - 13 Jan 2018

Any update here or someone can report a issue? If not i would be happy if that could be moved forward.

avatar zero-24 zero-24 - change - 18 Jan 2018
Labels Removed: ?
avatar Quy
Quy - comment - 18 Jan 2018

Please confirm that case #2 is not applicable here.

avatar zero-24
zero-24 - comment - 18 Jan 2018

Please confirm that case #2 is not applicable here.

hmm what do you mean by case #2 ?

avatar Quy
Quy - comment - 18 Jan 2018

How would this PR know that a.b.c.d (same ip address with web server) is local?

avatar zero-24
zero-24 - comment - 18 Jan 2018

How would this PR know that a.b.c.d (same ip address with web server) is local?

It should not be detect this as localhost ;)

Database host in following cases are considered as remote server (wrong detection)

avatar Quy Quy - test_item - 18 Jan 2018 - Tested successfully
avatar Quy
Quy - comment - 18 Jan 2018

I have tested this item successfully on 0026d06


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

avatar Quy
Quy - comment - 3 Feb 2018

@fastslack Can you please test? Thanks.

avatar zero-24
zero-24 - comment - 23 Jun 2018

@fastslack can you please take a look and check / test the patch again. So we can bring this finally into a release :)

avatar zero-24
zero-24 - comment - 28 Jul 2018

No one seams to be interested this days.. :(

avatar zero-24 zero-24 - change - 28 Jul 2018
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2018-07-28 11:07:25
Closed_By zero-24
avatar zero-24 zero-24 - close - 28 Jul 2018

Add a Comment

Login with GitHub to post a comment