Kegan Dougal
ca451098b2
Update httpcache to fix memory leak
8 years ago
Richard Lewis
4042324a40
Merge pull request #155 from matrix-org/google
Google image search bot
8 years ago
Richard Lewis
4e00142231
Merge branch 'google' of github.com:matrix-org/go-neb into imgur
8 years ago
Richard Lewis
07d27b67b1
Fix number of commands test
8 years ago
Richard Lewis
445c277bd3
Add default command route handler
8 years ago
Richard Lewis
4eeb46d6b9
Search for large images only. Fix httpClient reference. Fix logging when no error and no results
8 years ago
Richard Lewis
51b7f1ec5f
Imgur data model updates -- tests broken
8 years ago
Richard Lewis
6ba9cfb63e
Add cx paramater to sample config
8 years ago
Richard Lewis
b726158ac4
Fix message types
8 years ago
Richard Lewis
c46ad2e09b
Clean up named return value
8 years ago
Richard Lewis
c22c94b1c8
Cleaned up redundant variable
8 years ago
Richard Lewis
3ae13428d8
Moved google custom search engine ID in to service config. Cleaned up comments
8 years ago
Richard Lewis
333161d2e9
Improve command path parsing
8 years ago
Richard Lewis
19845fcc25
Redacted invalid auth keys. Comment / logging cleanup and formatting.
8 years ago
Richard Lewis
0f1ee35418
Check that query string is valid. Disable command execution test (currently broken)
8 years ago
Richard Lewis
b005c4fc83
Google image search bot
8 years ago
Kegsay
89056ebf9e
Merge pull request #152 from matrix-org/kegan/matrix-org-util
Use github.com/matrix-org/util
8 years ago
Kegan Dougal
3a7d5f84da
Bump requirements to Go 1.7 for net/http.Request.Context()
8 years ago
Kegan Dougal
cbc30321a4
Use github.com/matrix-org/util
8 years ago
Kegan Dougal
d6d19a8f6c
Add GH Webhook test
8 years ago
Kegsay
ae5b0df563
Merge pull request #147 from matrix-org/kegan/fix-log-field-names
Do not use log field names 'level', 'msg' or 'time'
8 years ago
Kegan Dougal
356ada4add
Do not use log field names 'level', 'msg' or 'time'
By default logrus uses these keys. If we also define one, logrus detects this
and then works around by adding 'fields.whatever' as the key name. However,
in the process of adding the key to the `Fields` it causes a race condition
which can crash the process with concurrent map r/w errors.
8 years ago
Kegsay
c7a127c2d8
Implement slackapi Service ( #116 )
8 years ago
Kegsay
5abfa281f5
Merge pull request #142 from matrix-org/kegan/gh-help
Add "!github help" and fix command selection bug
8 years ago
Kegan Dougal
96109afd41
Add "!github help" and fix command selection bug
The last command would always be chosen due to pointer fail.
8 years ago
Kegan Dougal
34ea2d06ce
Give a more helpful error message if a service is created with an unknown user ID
8 years ago
Kegsay
b1fe968f83
Merge pull request #137 from matrix-org/kegan/gomatrix
Use gomatrix
8 years ago
Kegan Dougal
8d75494c19
Also log when clients are created
8 years ago
Kegan Dougal
efe04d7917
Keep doing the Inviter hack when joining rooms so non-ops can kick the bot they invited
8 years ago
Kegan Dougal
e94ab0d18e
And log when we stop
8 years ago
Kegan Dougal
9708e6351b
Stop syncing on nil errors
8 years ago
Kegan Dougal
60429f32b3
Bug fixes
8 years ago
Kegan Dougal
b98e878227
Use gomatrix
8 years ago
Kegsay
2480163739
Merge pull request #135 from matrix-org/kegan/min-guid-length
Redo how recent GUIDs are calculated
8 years ago
Kegsay
9f77111702
Merge pull request #136 from matrix-org/kegan/feed-leak
Don't use gofeed.ParseURL
8 years ago
Kegan Dougal
7522ddf1a4
Explain why no ParseURL
8 years ago
Kegan Dougal
c4e98238d8
uniq prior to length checks for accuracy
8 years ago
Kegan Dougal
1722871345
Don't use gofeed.ParseURL
It leaks the response because it doesn't close the `resp.Body` on non-2xx.
8 years ago
Kegan Dougal
332b451d77
Redo how recent GUIDs are calculated
This is an attempt to fix #133 .
Previously, we just clobbered the recent GUIDs with the lastest response
every single time, assuming that Atom/RSS feeds would consistently return the
same items. This appears to not be the case. In the wild, the number of items
returned on a single request can vary (sometimes even being 1 or 2 when usually
it is 50!).
This patch alters *how many* and *which* GUIDs we keep between requests, in an
attempt to prevent sending old news for buggy RSS feeds.
8 years ago
Kegan Dougal
d128ed2626
Add more logging for RSS feeds
8 years ago
Kegsay
357abef22c
Merge pull request #134 from matrix-org/kegan/rss-dont-replace-if-no-guids
Treat RSS feeds with 0 items as errors
8 years ago
Kegan Dougal
677b556376
Return an error so it is logged
8 years ago
Kegan Dougal
66fb3253f7
Treat RSS feeds with 0 items as errors
In the wild it looks like some RSS feeds will occasionally return 0 items
to requests *but not return an error*. This previously meant we would clobber
our knowledge of recent GUIDs with the empty set. This meant that the next
successful poll would resend the **entire** RSS feed.
8 years ago
Kegsay
d7eb332948
Merge pull request #132 from matrix-org/kegan/giphy-translate-api
Use Giphy's translate API for better accuracy
8 years ago
Kegan Dougal
729d004ad0
Use Giphy's translate API for better accuracy
8 years ago
Kegsay
f2383a89ac
Merge pull request #131 from matrix-org/kegan/finer-gh-controls
Break apart 'issues' and 'pull_request' events
8 years ago
Kegan Dougal
aec58233d9
s/assign/assignments/
8 years ago
Kegan Dougal
4d918673c0
Break apart 'issues' and 'pull_request' events
Now broken down into:
- `labels` : Labelling/Unlabelling issues/PRs
- `milestones` : Milestoning/Demilestoning issues/PRs
- `assign` : Assigning/Unassigning issues/PRs
This is broken down in the guts of parsing the webhook event such that it
appears to be a unique `X-GitHub-Event` type.
8 years ago
Kegsay
402e29782a
Merge pull request #129 from matrix-org/kegan/testutils
Add a dedicated testutils package
8 years ago
Kegan Dougal
683d36b5c5
Review comments
8 years ago