* Set -x in the integration test
The integration test has been flaking from time to time on Travis with
no obvious failures, indicating this is happening unexpectedly outside a
`fail` call. `set -x` will help us diagnose when it happens again.
* Only show logs on test failure
* Correctly continue if test.sh fails
* Only cleanup the logfile on success. The logfile should stick around on
failure for easier sharing of test results. It will eventually get
cleaned up whenever the tmp directory gets cleaned.
This corresponds with the same change in outline-client
Tested:
yarn do {server_manager/test, shadowbox/test, shadowbox/integration_test/run, sentry_webhook/build, metrics_server/build}
* Allow the port fo rnew access keys to be changed from the UI
* Remove old TODOs
* Use correct log levels
* CSS formatting
* Use Number.isNan
* short circuit on no change and kill dead code
* Implement fakes, fix display bug
* fix typo
* Respond to code review
* Fix underline coloring
* Limit to numerical input
* Respond to review comments
* Review comments
* Disable editing key port for old servers (#513)
* Disable editing key port for old servers
* Use -1 instead of the current port number, nits
* Be more conservative with the response code check
* Use on-tap instead of on-click for the keys port input
* Rename "toggleFoo" to "setFoo"
* Use 1.0.0 instead of 0.2.0
* Respond to review comments. Also fixes a bug where, if the input was
invalid due to a server-side error, hitting Enter did nothing except
show a valid state
* Correct the previous bugfix
* Add an endpoint to set a new port for access keys
* post commit formatting
* Document the new api
* Remove enum in errors
* Respond to review comments
* Respond to review comments
* Rename to ShadowboxError and add copywright header
* Move to errors.ts
* Respond to coments in access_key.ts
* Move to errors.ts
* More renames
* post commit formatting
* More review responses
* Correctly convert invalid port number to string
* More renaming
* post commit formatting
* Respond to review comments
* Fix issue using parseInt
* Change endpoint URL
* formatting
* Respond to more review comments
* Respond to comments, add to integration test
* post commit formatting
* Move invalidPortArgument into a local
* formatting
* More review
* more review
* formatting
* disambiguate errors using restify types
* formatting
* fix the integration test
* Remove magic number responses from manager_service
* add 'new'
* De-nest branches in setShareMetrics
* Cleanup manager_service
* Fix error checking
* unformat
* Respond to review comments
Force merging because the travis tests passed but the github ui isn't updating.