| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
* Cleanup app state on initialization error
When returning an initialization error, the app state was not cleaned
up. This is especially visible during tests, as `appCount` is not
decremented, and makes the new app initialization fail.
* Test the `platform server` command
As the `platform server` command only exits when interrupted by
a signal, it is not possible to test it as the other cobra
commands. Instead we directly test the actual command function.
The internal command handler is slighly refactored to take
a channel in argument, and registers it as the signal handler.
Nothing very different—except than controlling this channel
from the outside allows the test to send the system signal
itself, thus preventing the server to run forever.
|
| | |
| |
| |
| |
| |
| |
| |
| | |
* remove license globals entirely
* fix infinite recursion
* test fix
|
| |\| |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
* [PTL-7537] implement feature and test
* [PTL-7537] Update feature to post the the room requiring a username flag to be used
* [PTL-7537] update tests with username
* update test to remove changes to the test helper struct
* use the basic team and user
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Only a handful of critical errors are present in the codebase.
They all occur during server startup (in `app.StartServer()`).
Currently, when one of these critical error occurs, it is simpled
mentionned in the logs – then the error is discarded, and the app
attempts to continue the execution (and probably fails pretty quickly in
a weird way).
Rather than continuing operations in an unknow state, these errors should
trigger a clean exit.
This commit rewrites critical startup errors to be correctly
propagated, logged, and then terminate the command execution.
Additionnaly, it makes the server return a proper error code to the
shell.
|
| | | |
|
| |\| |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
When starting the server using `platform server`, errors occuring during
startup are not reported in the console. The command exit with a 0 exit
code (i.e. "success"), although the server failed to launch.
With this change, when an error occurs while initializing the app (like
a missing or invalid configuration file):
- the error is printed to the console;
- the command exit with a "-1" exit code.
This allow shell scripts to properly detect the startup failure, and to
react to it.
Example of error displayed:
```
$ platform server
Error: LoadConfig: Error decoding config file=config.json, err=While parsing config: invalid character ':' after top-level value,
``
|
| | | |
|
| | | |
|
| |/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* XYZ-13: Update Permission and Role structs to new design.
* XYZ-10: Role store.
* XYZ-9/XYZ-44: Roles API endpoints and WebSocket message.
* XYZ-8: Switch server permissions checks to store backed roles.
* XYZ-58: Proper validation of roles where required.
* XYZ-11/XYZ-55: Migration to store backed roles from policy config.
* XYZ-37: Update unit tests to work with database roles.
* XYZ-56: Remove the "guest" role.
* Changes to SetDefaultRolesFromConfig.
* Short-circuit the store if nothing has changed.
* Address first round of review comments.
* Address second round of review comments.
|
| | |
|
| |
|
|
|
|
|
|
| |
* finally remove utils.Cfg
* fix compile error
* another test compilation fix
|
| |
|
|
|
|
| |
* Move session clean-up to daily task
* Split delete query into batches
|
| |
|
|
|
|
| |
* remove global cfg vars
* enterprise update
|
| |
|
|
|
|
|
|
|
|
| |
* remove global config watcher
* keep config watcher disabled for tests
* compile fix
* fix resource leak
|
| |
|
|
|
|
|
|
| |
* Add fake dependency
* [XYZ-6] Add sampledata platform command
* Creating EMOJI_NAME_MAX_LENGTH as a constant and using it where needed
|
| | |
|
| |
|
|
|
|
| |
* remove utils.ClientCfg and utils.ClientCfgHash
* remove unused import
|
| | |
|
| |
|
|
|
|
|
|
| |
* add plugin slash command support
* remove unused string
* rebase
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Created message export scheduled task
* Added CLI command to immediately kick off an export job
* Added email addresses for users joining and leaving the channel to the export
* Added support for both MySQL and PostgreSQL
* Fixing gofmt error
* Added a new ChannelMemberHistory store and associated tests
* Updating the ChannelMemberHistory channel as users create/join/leave channels
* Added user email to the message export object so it can be included in the actiance export xml
* Don't fail to log a leave event if a corresponding join event wasn't logged
* Adding copyright notices
* Adding message export settings to daily diagnostics report
* Added System Console integration for message export
* Cleaned up TODOs
* Made batch size configurable
* Added export from timestamp to CLI command
* Made ChannelMemberHistory table updates best effort
* Added a context-based timeout option to the message export CLI
* Minor PR updates/improvements
* Removed unnecessary fields from MessageExport object to reduce query overhead
* Removed JSON functions from the message export query in an effort to optimize performance
* Changed the way that channel member history queries and purges work to better account for edge cases
* Fixing a test I missed with the last refactor
* Added file copy functionality to file backend, improved config validation, added default config values
* Fixed file copy tests
* More concise use of the testing libraries
* Fixed context leak error
* Changed default export path to correctly place an 'export' directory under the 'data' directory
* Can't delete records from a read replica
* Fixed copy file tests
* Start job workers when license is applied, if configured to do so
* Suggestions from the PR
* Moar unit tests
* Fixed test imports
|
| |
|
|
|
|
|
|
| |
* role refactor
* add missing file
* fix web test
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
team (#7752)
* Fixes system message posted to Town Square when adding/removing team member (#6483)
Fixes unit test method call to match RemoveUserFromTeam modification (#6483)
Fixes system message posted to Town Square when adding/removing team member (#6483)
Removes unnessary error message
* Updates system message when a user leaves a team (#6483)
* Changes system message when a user is removed from team (#6483)
* Adds a new string to be post to town-square when a member is added to a team (#6483)
|
| | |
|
| |
|
|
|
|
|
|
| |
* reorganize file util functionality
* fix api test compilation
* fix rebase issue
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
|
| |
* improve test coverage reporting / accuracy
* handle absolute coverpaths
* move tests into multiple files
* rename codecov.yml (https://github.com/codecov/support/issues/426)
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
|
| |
* PLT-8011: Make Elasticsearch startup fully async.
* Use a.Go
|
| |
|
|
|
|
|
|
| |
* Fix plugin config watcher for enabling and disabling
* Make sure HTTP routes are served
* Set content-type explicitly
|
| | |
|
| |
|
|
|
|
|
|
| |
* Add websocket event for user role update
* Fix tests
* More test fixes
|
| |
|
|
|
|
| |
* refactoring cfg refs and load / save functions
* improve error output
|
| |
|
|
|
|
| |
* remove unused variables / code
* fix bug in testPostStoreGetOldest
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
| |
Fix the "team create" command wrongly shows a `teams create` example, where an extra "s" is appended to the command name.
|
| |
|
|
|
|
|
|
|
|
|
|
| |
* use tmpfs containers for api/api4, move and speed up cli tests
* minor optimizations
* add missing files, fix pre-existing race condition
* add . to TestMain check
* add requested log message
|
