ae8900cc69
Merge pull request #259 from pkgw/polish
...
More fixing
2020-12-10 14:35:41 -05:00
5cef2f8474
WWT.Web: make sure to propagate query string and fragment when constructing request URI
...
Should fix ShowImage.aspx issues.
2020-12-10 14:22:45 -05:00
c8eef87abc
Modernize some webclient URLs and fix ShowImage URL manipulation
...
We were getting 404's for `ShowImage.aspx&wtml=true` because the code
was just appending the `&wtml=true` brainlessly.
2020-12-10 14:05:02 -05:00
87371adad0
WWT.Providers: MarsDEMProvider: simplify, given the reality that the chunk files are gone
2020-12-10 13:31:30 -05:00
9425252e72
Merge pull request #258 from pkgw/deprecations
...
Deprecations
2020-12-10 13:07:24 -05:00
88074939e7
docs/deprecations.md: log the newly deprecated APIs
2020-12-10 12:54:32 -05:00
7ee58eb9a7
docs/deprecations.md: move list of deprecated APIs from README to here
2020-12-10 12:49:09 -05:00
fe49afbc2a
WWT.Providers: deprecate unused/broken APIs
...
Closes #204 .
2020-12-10 12:48:20 -05:00
3cf10bea03
Merge pull request #257 from pkgw/renames
...
WWT.Providers: bulk-rename files
2020-12-10 12:19:08 -05:00
11ee3c0a87
WWT.Providers: bulk-rename files
...
No code changes
2020-12-10 12:07:57 -05:00
960552d907
Merge pull request #256 from pkgw/misc-fixes
...
Miscellaneous fixes from rollout
2020-12-10 11:41:45 -05:00
9278869e3a
WWT.Providers: update providers using 4-item Q parameters
2020-12-10 11:25:39 -05:00
2f8b30bed2
WWT.Providers: centralize parsing of Q parameters
...
This commit leaves out a few endpoints that process a fourth "Q"
argument. Now we will consistently report a sensible error if one of
the Q inputs is not parseable as an integer!
2020-12-10 11:20:10 -05:00
469cb3c619
WWTMVC5: attempt to fix URLs exposed in /Resource endpoints
...
As per #250 , we need to make it possible to configure the hostname that
will be exposed to the world, since our server will be running behind a
gateway with a non-public hostname.
2020-12-10 09:55:39 -05:00
83e000268c
WWTMVC5: remove all-commented WebServices/ResourceService.svc.cs
2020-12-10 09:33:18 -05:00
8c11af515f
Merge pull request #255 from pkgw/logging-mop-up
...
WWT.Caching: turn a common LogInformation into a LogTrace
2020-12-09 12:52:33 -05:00
3b6775e12d
WWT.Caching: turn a common LogInformation into a LogTrace
2020-12-09 12:38:37 -05:00
baa6ea1653
Merge pull request #254 from pkgw/more-logging
...
More iteration on the Linux server
2020-12-09 12:12:15 -05:00
fcadd27798
src/WWT.Web/appsettings.json: attempt #2 to reduce App Insights traffic
2020-12-09 11:56:02 -05:00
3fbaccb123
Dockerfile: set ASP.NET Core environment to Production
2020-12-09 11:54:58 -05:00
683355f873
Merge pull request #253 from pkgw/scripts-wwtsdk-aspx
...
Try to provide /Scripts/wwtsdk.aspx
2020-12-09 10:25:55 -05:00
d25b046866
azure-pipelines.yml: clean app service deployment destination
...
Hopefully this will prevent recurrences where our deployment gets busted but we don't
notice because the old files just stick around.
2020-12-09 10:12:29 -05:00
082e8f9563
WWTMVC5: import the wwtsdk.aspx script
...
One of our more common errors is related to the lack of this file.
2020-12-09 10:10:10 -05:00
87cb51617d
src/WWTMVC5/WWTMVC5.csproj: work on the ordering and grouping of the items in here
2020-12-09 10:08:55 -05:00
87cc31aae1
Merge pull request #251 from pkgw/less-logging
...
src/WWT.Web/appsettings.json: try to reduce App Insights telemetry spam
2020-12-09 09:40:57 -05:00
5154312956
src/WWT.Web/appsettings.json: try to reduce App Insights telemetry spam
2020-12-09 09:24:32 -05:00
f9856b35aa
azure-pipelines.yml: deploy MVC5 webapp from a Windows host, not Linux
...
I think this might fix our deployment problems.
2020-12-08 11:27:05 -05:00
05d1807a83
src/WWTMVC5/Views/Shared/_Layout.cshtml: add a comment to see if our app is actually deploying
2020-12-08 10:50:03 -05:00
07a68c9192
Merge pull request #248 from pkgw/oauth-again
...
WWTMVC5: oops, OAuth redirect change was only half-baked
2020-12-07 23:59:29 -05:00
2e8aa7190d
WWTMVC5: oops, OAuth redirect change was only half-baked
2020-12-07 23:50:28 -05:00
76e00c8a9b
Merge pull request #247 from pkgw/robustness
...
Robustness
2020-12-07 22:59:24 -05:00
f8ed80f293
Attempt to fix OAuth redirection with deployment slots
...
It turns out that if we're going to have "stage" and "production"
deployment slots that we're going to hot-swap, they need to have the
same ASP.Net app settings. (This makes sense in retrospect since if they
had different settings, you'd have to change the settings on-the-fly
during the swap, which isn't possible.) So the method that we had for
setting the OAuth redirection URLs isn't tenable. Instead, we need to
configure a dictionary of potential URLs and choose one at runtime based
on the HTTP Host header.
2020-12-07 21:37:39 -05:00
64462ba3d6
WWT.Providers: body with HTTP 304 is apparently disallowed
2020-12-07 20:05:51 -05:00
fff35839cb
Merge pull request #246 from pkgw/aspcore-unimplementeds
...
Avoid APIs unimplemented in ASP.Net Core
2020-12-07 15:10:12 -05:00
365302c45d
src/WWT.Web/HelloWorldProvider.cs: use recommended InformationalVersion attr
2020-12-07 14:59:06 -05:00
3296a82d07
azure-pipelines.yml: update deployment remarks
2020-12-07 14:55:22 -05:00
cee087c928
Merge pull request #245 from pkgw/tokens
...
Mop up some awaits that don't pass around the cancellation token
2020-12-07 13:42:03 -05:00
304be624eb
WWT.Providers: add some comments about wonky content-types
2020-12-07 13:40:53 -05:00
2f900ba6b5
Remove unused IResponse.CacheControl
2020-12-07 13:00:30 -05:00
b85a218add
Remove and replace IResponse.WriteFile
2020-12-07 12:59:18 -05:00
c352485071
Remove IWwtContext.MapPath -- unused and unimplemented in ASP.Net Core
2020-12-07 12:08:49 -05:00
1473432085
Remove IRequest.PhysicalPath -- unused and unimplemented in ASP.Net Core
2020-12-07 12:07:49 -05:00
80a475aa06
Remove IResponse.Expires
...
This was also unimplemented in the ASP.Net Core server. We replace it
with a manual setting of the Expires header. Here I also remove a few
providers that manually set Expires/ETag/Last-Modified headers in some
way that can't possibly be The Right Way.
2020-12-07 12:04:11 -05:00
af8fa70901
Remove IResponse.Status, unimplemented in ASP.NET Core, and replace
2020-12-07 11:55:21 -05:00
57e4fc1099
Mop up some awaits that don't pass around the cancellation token
...
My understanding is that you should always propagate the token if
possible. Fixing this may fix occasional TaskCancelledExceptions that
we see in the App Insights, including in the workhorse DSS provider.
2020-12-06 18:42:16 -05:00
ed5eb546ea
Merge pull request #244 from pkgw/robots933456
...
Avoid some annoying 404's in the WWT.Web server
2020-12-06 17:18:01 -05:00
4ad3b81730
Avoid some annoying 404's in the WWT.Web server
...
It turns out that the Azure Docker service checks a URL named
/robots933456.txt to test whether the container is ready to handle
requests. The service doesn't care if its requests yield 404s, but those
404's do get logged to App Insights, which we'd prefer to avoid.
While we're at it, we stop returning 404's for requests to "/", since
many web infrastructure pieces assume that your app will return a
success code for requests to the root (including App Gateway's health
checks).
2020-12-06 17:04:30 -05:00
bcc3317c5a
Merge pull request #243 from pkgw/deploy-notes
...
Document the deployment processes a bit
2020-12-06 14:51:48 -05:00
998fd0a48f
Document the deployment processes a bit
2020-12-06 14:41:14 -05:00
18d2b0f2b1
Merge pull request #242 from pkgw/yet-more-linux
...
Yet more Linux fixing
2020-12-04 19:46:54 -05:00
Peter Williams