Azure / wordpress-linux-appservice

MIT License
113 stars 71 forks source link

Frontdoor, website breaking when i set home page #77

Closed The-Real-Adeel closed 1 year ago

The-Real-Adeel commented 1 year ago

Hi, I got the deployment working with front door.

However, when I do the most common practice of changing the home page to a specific page: Screenshot_20230426_124940_Outlook~2

The website breaks, with an error too many redirects: Screenshot_20230426_125016_Outlook~3

Which tells me its a problem with the deployment itself (either the container or missing elements on front door)

What can be done to address this?

rabollin commented 1 year ago

@The-Real-Adeel - Working on this issue and will get back at the earliest.

The-Real-Adeel commented 1 year ago

Thank you. Looking forward to your response

rabollin commented 1 year ago

Hi - I think the issue is resolved now, can you please confirm if we can close the issue.

The-Real-Adeel commented 1 year ago

Hi @rabollin, Are you referring to my communication with your colleague Sai? Sorry I was out of office last week just came back to test the suggestions provided by them.

Unfortunately it is not a fix. It fixed the problem but opened up new ones. ie, can access home page without redirects but not others pages like admin page.

I have messaged him directly in teams, not sure if that is the correct protocol to submit a ticket. Would like another remote session to showcase the challenges I am facing now.

As for the official deployment has there been changes made? I haven't redeployed just edited the wp-config. I would prefer to have the deployment itself fleshed out with AFD as we plan to deploy many sites in the near future. Altering wp-config each time manually is unsustainable for us.

The-Real-Adeel commented 1 year ago

Just want to follow up @rabollin , I touched base with Sai today and he was able to show me my mistake and get it working!

It seems to work as intended now. Thank you guys for the support.

However, please keep this issue open until there is an update on the deployment itself to include this fix. I would like to be alerted when that happens.

rabollin commented 1 year ago

@The-Real-Adeel - Issue is fixed now permanently and if you create a new site from Marketplace offering, you will not run into this issue. Please let us know if you have any other questions before we close this issue.

The-Real-Adeel commented 1 year ago

@rabollin Awesome, thats great to hear!

I actually use terraform for deployment for this. Do you have any insight on what would I need to be changed in the deployment scripts? ie, New image to point to?

Our deployment is going to use an existing AFD which has a custom domain. Just curious what needs to be altered.

The-Real-Adeel commented 1 year ago

Hi @rabollin

I also just did a deployment from the marketplace as a test in my tenant (to see the changes to gain insight from). It looks like am getting the same error when I set a the homepage to a specific page and set permalinks to posts on WP.

Here are some screenshots for this for your insight on what came out when I used this:

image Same error

image Container Version

image Whats inside wp-config file. It doesn't include the fix suggested by your colleague

image THE AFD entries that were added

edit: Please note, I still have a work around assisted by your colleague which works but requires a manual step (changes to the wp-config file). I am waiting on THOSE changes to the official deployment to address this manual step which he just informed me is still in the works.

rabollin commented 1 year ago

@The-Real-Adeel - Thank you for your patience in verifying this issue multiple times, the issue is fully resolved now and the new deployments from Marketplace will have working solution. Can you please validate once and confirm if we can close the issue.

The-Real-Adeel commented 1 year ago

Validated. it is indeed working!! Thank team for your work.