Rewrite rules are not working on production

I created a bunch of rewrite rules to make sure that old .php URLs still work.

However, while I was doing some testing today, I've noticed that these no longer work: https://gitlab.eclipse.org/eclipsefdn/it/websites/eclipse.org/-/blob/main/config/nginx/default.conf#L50

For example, https://www.eclipse.org/home/search.php should be redirect to https://www.eclipse.org/home/search/

From what I can tell, the Nginx server config are properly being configured: https://gitlab.eclipse.org/eclipsefdn/it/websites/eclipse.org/-/blob/main/Jenkinsfile#L6

@mward @mbarbero Any thoughts on why this is not working?

added team:releng team:webdev labels

assigned to @cguindon, @mbarbero, and @mward

added priorityhigh severitymajor statetodo labels

I am putting this a high priority cause some functionality on the site is not working properly because of it. For example, the search is broken on some pages: https://www.eclipse.org/home/search.php?q=fsasdf

@mbarbero I took a look at a docker container that we publish for the site and the content of my default.conf is not included. I see our default file which does not include my redirect rules...

@fgurr Since Mika is on vacation, do you know why nginxServerConf is being ignored? It appears to be a valid config based on what I see on: https://gitlab.eclipse.org/eclipsefdn/it/releng/jenkins-pipeline-library/-/blob/main/vars/hugo.groovy

added statewip label and removed statetodo label

assigned to @fgurr and unassigned @mward

Removing @mward. After my initial investigation, this appears to be a releng (ci/cd) issue.

@fgurr Since Mika is on vacation, do you know why nginxServerConf is being ignored? It appears to be a valid config based on what I see on: https://gitlab.eclipse.org/eclipsefdn/it/releng/jenkins-pipeline-library/-/blob/main/vars/hugo.groovy

Not at a first glance. I would have to dig in deeper next week. Since Mikael will be back by then, I guess this becomes void.

Sounds good. I will reach out to @mbarbero about this. I am going to assume that functionality hasn't been deployed yet.

unassigned @fgurr

Fixed with 83df29f8

added statereview label and removed statewip label

@mbarbero I think we are done here?

Yes, we're done. I did not close this one just to give you an opportunity to review the fix.

Thanks, it seems like it's working like expected now! I will re-open if I find any new issues!

closed