I guess that is an unfortunate accident in this case. But man it was frustrating, I even had some co workers who are nginx experts and they missed it to.
It is certainly embarrassing. Thanx for your reply. Julian On Fri, Apr 26, 2019 at 10:11 AM Francis Daly <[email protected]> wrote: > On Thu, Apr 25, 2019 at 08:08:47PM -0500, Julian Brown wrote: > > Hi there, > > Well done in spotting that -- it has come up before, but obviously wasn't > something that someone noticed quickly enough this time. > > > I finally figured it out, and I cannot believe it passed a syntax > checker. > > It passed a syntax check because it is syntactically valid. > > > server { > > server_name learngigs.com www.learngigs.com > > > > listen 443; > > listen [::]:443; > > It is inconvenient in this case; but "server_name" accepts a list > of whitespace-separated arguments. Your "server_name" directive has > four arguments. > > It's not what you intended, but the computer (in general) does not care > what you intended; it cares what you wrote. > > > I did not have a semi colon after the server name directive. The syntax > > checker said it was fine, so I do not know what it was trying to do. > > I suspect that this won't be "fixed", because the amount of special-casing > required in the code to handle it is probably not worth the effort to > anyone to write. > > For example: a week ago, you would have liked if someone had previously > come up with a reliable and obviously-documented way that this specific > problem could be auto-avoided or -alerted. Today, you probably don't > need that work done, because you will remember to check for semi-colon > if you ever see the same problem again. > > Great that you found and fixed the problem in the config! > > Cheers, > > f > -- > Francis Daly [email protected] > _______________________________________________ > nginx mailing list > [email protected] > http://mailman.nginx.org/mailman/listinfo/nginx >
_______________________________________________ nginx mailing list [email protected] http://mailman.nginx.org/mailman/listinfo/nginx
