summaryrefslogtreecommitdiffstats
path: root/app/webhook.go
Commit message (Collapse)AuthorAgeFilesLines
* added the custom icon and username for the outgoing webhook and its response ↵Pradeep Murugesan2018-07-251-0/+6
| | | | | | | | | | | | (#9141) * 8272 added the username and icon as part of the model and persisted the same * 8272 added the custome icon and name when set to the web hook response * 8272 changed the infinte loop to timeout after 5 seconds * 8272 fixed review comments
* MM-11106: Allow systeadmin webook to post to read only town square. (#9051)George Goldberg2018-07-061-1/+10
|
* MM-10352: Add locking incoming webhooks to a single channel. (#8835)George Goldberg2018-05-221-0/+4
|
* Structured logging (#8673)Christopher Speller2018-04-271-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Implementing structured logging * Changes to en.json to allow refactor to run. * Fixing global logger * Structured logger initalization. * Add caller. * Do some log redirection. * Auto refactor * Cleaning up l4g reference and removing dependancy. * Removing junk. * Copyright headers. * Fixing tests * Revert "Changes to en.json to allow refactor to run." This reverts commit fd8249e99bcad0231e6ea65cd77c32aae9a54026. * Fixing some auto refactor strangeness and typo. * Making keys more human readable.
* Relax 4k post message limit (#8478)Jesse Hallam2018-03-261-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * MM-9661: rename POST_MESSAGE_MAX_RUNES to \0_v1 * MM-9661: s/4000/POST_MESSAGE_MAX_RUNES_V1/ in tests * MM-9661: introduce POST_MESSAGE_MAX_RUNES_V2 * MM-9661: migrate Postgres Posts.Message column to TEXT from VARCHAR(4000) This is safe to do in a production instance since the underyling type is not changing. We explicitly don't do this automatically for MySQL, but also don't need to since the ORM would have already created a TEXT column for MySQL in that case. * MM-9661: emit MaxPostSize in client config This value remains unconfigurable at this time, but exposes the current limit to the client. The limit remains at 4k in this commit. * MM-9661: introduce and use SqlPostStore.GetMaxPostSize Enforce a byte limitation in the database, and use 1/4 of that value as the rune count limitation (assuming a worst case UTF-8 representation). * move maxPostSizeCached, lastPostsCache and lastPostTimeCache out of the global context and onto the SqlPostStore * address feedback from code review: * ensure sqlstore unit tests are actually being run * move global caches into SqlPostStore * leverage sync.Once to address a race condition * modify upgrade semantics to match new db semantics gorp's behaviour on creating columns with a maximum length on Postgres differs from MySQL: * Postgres * gorp uses TEXT for string columns without a maximum length * gorp uses VARCHAR(N) for string columns with a maximum length of N * MySQL * gorp uses TEXT for string columns with a maximum length >= 256 * gorp uses VARCHAR(N) for string columns with a maximum length of N * gorp defaults to a maximum length of 255, implying VARCHAR(255) So the Message column has been TEXT on MySQL but VARCHAR(4000) on Postgres. With the new, longer limits of 65535, and without changes to gorp, the expected behaviour is TEXT on MySQL and VARCHAR(65535) on Postgres. This commit makes the upgrade semantics match the new database semantics. Ideally, we'd revisit the gorp behaviour at a later time. * allow TestMaxPostSize test cases to actually run in parallel * default maxPostSizeCached to POST_MESSAGE_MAX_RUNES_V1 in case the once initializer panics * fix casting error * MM-9661: skip the schema migration for Postgres It turns out resizing VARCHAR requires a rewrite in some versions of Postgres, but migrating VARCHAR to TEXT does not. Given the increasing complexity, let's defer the migration to the enduser instead.
* [MM-9725] Slack compatibility code in webhooks captures incorrect text (#8428)Carlos Tadeu Panato Junior2018-03-121-1/+1
|
* add App.License, remove utils.IsLicensed / utils.License calls (#8203)Chris2018-02-061-1/+1
|
* PLT-3658 Added support for Slack attachments in outgoing webhook response ↵Joey Lee2018-01-261-2/+13
| | | | (#7774) (#8102)
* [PLT-8173] Strip the post_ prefix on incoming webhook overrides. (#8019)Jesse Hallam2018-01-031-10/+10
|
* [PLT-8173] Add username and profile picture to webhook set up pages (#8002)Jesse Hallam2018-01-021-2/+31
|
* [PLT-8438] Include incoming webhook display name in post props (#7997)Carlos Tadeu Panato Junior2017-12-221-11/+17
|
* HTTP client refactor (#7884)Chris2017-11-221-1/+1
| | | | | | * http client refactor * simplification
* [PLT-840] Add option to outgoing webhooks to reply to the posted message as ↵Carlos Tadeu Panato Junior2017-11-201-4/+9
| | | | a comment (#7807)
* PLT-7824 Added support for mentions with <@userid> and <!here> (#7615) (#7737)Joey Lee2017-11-171-0/+3
|
* simplify things (#7735)Chris2017-10-301-5/+2
|
* PLT-7599: webhook post splitting (#7707)Chris2017-10-241-32/+86
| | | | | | | | * webhook post splitting * style fix * update old webhook test
* Reduce utils.Cfg references (#7650)Chris2017-10-181-19/+19
| | | | | | | | | | | | * app.UpdateConfig method * test fix * another test fix * the config override option as-was is just error prone, remove it for now * derp
* Merge branch 'release-4.3'Christopher Brown2017-10-161-0/+5
|\
| * PLT-7818 Updates to post type (#7579)Joram Wilander2017-10-091-0/+5
| | | | | | | | | | | | * Updates to post type * Update tests
* | Add back consumeAndClose functionality. (#7608)Christopher Speller2017-10-121-1/+1
| | | | | | | | | | | | | | * consume bodies for action button integrations, webrtc gateway, oauth endpoint * Fixing a couple more places, switching to io.Copy to ioutil.Discard, adding a comment to help prevent future performance regressions
* | Miscellaneous app cleanup (#7594)Chris2017-10-091-1/+1
|/ | | | | | | | | | | | * app cleanup * whoops, forgot a file * some minor cleanup * longer container deadline * defensive checks
* Goroutine wranglin (#7556)Chris2017-10-031-14/+20
| | | | | | * goroutine wranglin * synchronize WebConn.WritePump
* PLT-7684 Add support to outgoing webhooks and slash commands to set post ↵Joram Wilander2017-09-281-3/+3
| | | | | | | | type and props (#7531) * Add support to outgoing webhooks and slash commands to set post type and props * Fix nil access
* Finally do away with NewLocAppError function. (#7498)George Goldberg2017-09-251-1/+1
| | | | | This cleans up the few NewLocAppError calls that crept in since the main bulk of them were removed, and finally removes the NewLocAppError function altogether.
* remove einterface gets (#7455)Chris2017-09-191-2/+1
|
* Invalidate webhook cache after updating webhook (#7430)Joram Wilander2017-09-141-0/+1
|
* Renaming repoChristopher Speller2017-09-061-4/+4
|
* app type transition (#7167)Chris2017-09-061-56/+56
|
* App: NewLocAppError -> NewAppError (#7327)George Goldberg2017-09-011-3/+3
| | | | | | * App: NewLocAppError -> NewAppError * Remove statuscode that got missed.
* Config to make town square read only (#7140)David Meza2017-09-011-0/+5
| | | | | | | | | | | | | | | | | | | | | | * Be able to make Town Square read-only (Disable typing messages for non admins). * Do not emit UserTypingEvent when TownSquareIsReadOnly and is Town Square. * Add unit tests for TownSquareIsReadOnly config value and logic. * Add TownSquareIsReadOnly to System console>Policy. Added Telemetry. * Add control for TownSquareIsReadOnly=true only for License Enterprise Edition E10 & E20. * Update en.json * Update en.json * Update policy_settings.jsx * Change config value from TownSquareIsReadOnly to ExperimentalTownSquareIsReadOnly. * Refactored to simplify. Avoid code repeat and multiple db calls.
* PLT-5745 Cleaning up duplicate calls (#7298)Corey Hulen2017-08-281-7/+7
| | | | | | | | * PLT-5745 Cleaning up duplicate calls * Fixing missing calls * Fixing broken test
* PLT-1649: add response_url support for custom slash commands (#6739)Chris2017-08-161-0/+51
| | | | | | | | | | * add response_url support for custom slash commands * pr suggestions * pr update / suggestion * test fix
* PLT-6358: Server HTTP client improvements (#6980)Chris2017-08-091-1/+1
| | | | | | | | | | | | | | | | * restrict untrusted, internal http connections by default * command test fix * more test fixes * change setting from toggle to whitelist * requested ui changes * add isdefault diagnostic * fix tests
* PLT-6965 jira integration (plus plugin scaffolding) (#6918)Chris2017-08-021-21/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * plugin scaffolding / jira integration * add vendored testify packages * webhook fix * don't change i18n ids * support configuration watching * add basic jira plugin configuration to admin console * fix eslint errors * fix another eslint warning * polish * undo unintentional config.json commit >:( * test fix * add jira plugin diagnostics, remove dm support, add bot tag, generate web-safe secrets * rebase, implement requested changes * requested changes * remove tests and minimize makefile change * add missing license headers * add missing comma * remove bad line from Makefile
* [PLT-6676] Make OutgoingWebhook to fire when post has no text content but ↵Saturnino Abril2017-07-181-50/+56
| | | | | | | | only attachment (#6935) * make OutgoingWebhook to fire when post has no text content but only attachment * update per comment and modify payload & test
* PLT-6581 breakup webhooks greater than 4K into multiple posts (#6530)Corey Hulen2017-06-021-8/+29
|
* PLT-5705 Created a single source of http.Client creation logic with internet ↵Torsten Juergeleit2017-05-311-11/+2
| | | | proxy support, reasonable timeouts and optional insecure connections (#6503)
* PLT-5860 Updated copyright date (#6058)Harrison Healey2017-04-121-1/+1
| | | | | | | | | | | | * PLT-5860 Updated copyright date in about modal * PLT-5860 Updated copyright notice in JSX files * PLT-5860 Updated copyright notice in go files * Fixed misc copyright dates * Fixed component snapshots
* PLT-5900 Removed automatic configuration of SiteURL (#5952)Harrison Healey2017-04-041-6/+6
| | | | | | | | | | | | | | | | * Revert "PLT-5765 Passed SiteURL to SendNotifications (#5705)" This reverts commit a8e68bd8905972ae59de90fa33d5b3e3c274dc47. * Removed automatic configuration of SiteURL * Reverted unintentional config changes * Updated help text * Added link to docs in Site URL warning * Fixed merge conflict
* Merge branch 'release-3.7' into master.George Goldberg2017-03-171-38/+9
|\
| * PLT-5800 Cleaned up duplicated code for adding slack attachments to posts ↵Harrison Healey2017-03-101-32/+3
| | | | | | | | (#5711)
| * PLT-5765 Passed SiteURL to SendNotifications (#5705)Harrison Healey2017-03-101-6/+6
| |
* | cannot change ID and deleteat when trying to update the hook (#5784)Carlos Tadeu Panato Junior2017-03-161-0/+1
| |
* | Implement GET and POST /hooks/outgoing endpoints for APIv4 (#5645)Joram Wilander2017-03-131-4/+28
|/
* Move webhook logic from api layer to app layer (#5527)Joram Wilander2017-02-281-5/+298
| | | | | | | | * Move webhook logic from api layer to app layer * Consolidate error messages * Fix permission check and unit test
* Adding edit of incoming webhook (#5272)Poornima2017-02-261-1/+1
| | | | | | | Adding edit of outgoing webhook Fixing spelling of error Fixing style Changing from PUT to POST for updates Fixing test failures due to merge
* Implement create and get incoming webhook endpoints for APIv4 (#5407)Joram Wilander2017-02-211-0/+39
| | | | | | | | * Implement POST /hooks/incoming endpoint for APIv4 * Implement GET /hooks/incoming endpoint for APIv4 * Updates per feedback
* Adding webhook metrics (#5348)Christopher Speller2017-02-101-0/+5
|
* Refactor and migrate more functions out of api into app package (#5063)Joram Wilander2017-01-131-3/+37
|
* PLT-4938 Add app package and move logic over from api package (#4931)Joram Wilander2017-01-131-0/+155
* Add app package and move logic over from api package * Change app package functions to return errors * Move non-api tests into app package * Fix merge