Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Kazoo 304 #38

Merged
merged 22 commits into from
May 15, 2013
Merged

Kazoo 304 #38

merged 22 commits into from
May 15, 2013

Conversation

jamesaimonetti
Copy link
Member

Adds a token bucket per account/IP combo for throttling queries (returns a 429 Too Many Requests).
Moves crossbar bindings into an ETS table for concurrent reading (I think we can filter the result set before actually processing which match, but that's for another day).
Fix how we determine which nouns in the URL match a crossbar module (don't hit the file system to check for a beam for each path token).

Crossbar is much faster per request, and much much faster on concurrent requests (when not hitting a view).

James Aimonetti added 22 commits May 14, 2013 18:00
KAZOO-304: add token bucket infrastructure to crossbar
9b79963 
Add the crossbar_module_sup back to the mix, for cb_* modules to start their
supervisors under;
cb_buckets_sup is the supervisor for managing the ets table workers and the
kz_token_bucket supervisor;
cb_kz_buckets_sup is the sofo supervisor for the kz_token_bucket servers;
cb_buckets_mgr is the owner and heir of the ETS table, and passes control to the
server used for finding bucket servers
cb_buckets_ets is the interface for reading/creating bucket servers, based on
account_id and client_ip.
KAZOO-304: more strict checking when spawning token bucket server
73fd423
KAZOO-304: accessor for client_ip from cb_context
538189e
KAZOO-304: allow {'halt', Context} as a return from binding server's map
bf2531d
KAZOO-304: start the crossbar_module_sup on startup
de9e52a
KAZOO-304: update skel with formatting, specs
1fffe84
KAZOO-304: add token bucket check to cb_token_auth, before checking t…
5e58b91 
…oken auth
KAZOO-304: fix languages_provided return
3586110
KAZOO-304: check for the halt tuple from map and halt processing if f…
a9d4399 
…ound
KAZOO-304: add log to startup of token bucket
f0308ac
KAZOO-304: logging of token counts by account/ip
08a81e7
KAZOO-304: fetch config value with correct key
75e1368
KAZOO-304: tickies in cb_context.erl
8aee86b
KAZOO-304: move bindings into ets table
0fd4e94
KAZOO-304: open doc using open_cache_doc
859243c
KAZOO-304: allow req if token bucket server isn't up yet
6444b25
KAZOO-304: stop the token bucket supervisor
b6b62d5
KAZOO-304: add logging to v1_resource
798c41c
KAZOO-304: parse_path_tokens doesn't check the filesystem
c19fe69 
We try to call cb_module:module_info() to see if the cb_* module exists
rather than checking the filesystem for the beam file
KAZOO-304: tickies for couch_mgr
32d53ea
KAZOO-304: add clauses to help with undefined account dbs
03969b5
KAZOO-304: update fax with cowboy change
b60a009
k-anderson added a commit that referenced this pull request May 15, 2013
@k-anderson
Merge pull request #38 from 2600hz/KAZOO-304
b7a85d1 
Kazoo 304
@k-anderson k-anderson merged commit b7a85d1 into master May 15, 2013
@k-anderson k-anderson deleted the KAZOO-304 branch June 5, 2014 23:28
jamesaimonetti added a commit that referenced this pull request Sep 11, 2018
@jamesaimonetti
# This is a combination of 56 commits.
8ae498f 
# This is the 1st commit message:

Merge the diff into the current doc and try saving again

cleanup logging

prefer the diff's value (excepting _rev)

# This is the commit message #1:

simplfy doc return

# This is the commit message #2:

only convert to binary if a list of ascii is present

also don't convert kz_json:object() to null if found

update tests accordingly

# This is the commit message #3:

use sets:subtract/2 to find the diff in J1 and J2

# This is the commit message #4:

add ability to merge list of jobjs with strategy fun

# This is the commit message #5:

more diff testing

# This is the commit message #6:

flag ensure_saved as deprecated

# This is the commit message #7:

remove arity-4 export

we can change arity-3 to accept options instead of the updates directly

# This is the commit message #8:

ignore return

# This is the commit message #9:

refactor update_doc to take an options list instead

# This is the commit message #10:

update account docs with update proplist

# This is the commit message #11:

add specs and update type

# This is the commit message #12:

update usage of kz_datamgr:update_doc

# This is the commit message #13:

update uses of update_doc

# This is the commit message #14:

expose paths for properties

also fixes updating the version in the accounts DB

# This is the commit message #15:

expose the base properties being set

# This is the commit message #16:

use a setter

# This is the commit message #17:

expose more paths

# This is the commit message #18:

update uses of ensure_saved to use update_doc

# This is the commit message #19:

fix function call

# This is the commit message #20:

ignore returns

# This is the commit message #21:

less args passed around

# This is the commit message #22:

filter undefined

# This is the commit message #23:

takes a flat proplist

# This is the commit message #24:

match return

# This is the commit message #25:

use the saved version in future contexts

# This is the commit message #26:

just try to delete the doc without opening it

# This is the commit message #27:

handle returns

# This is the commit message #28:

handle return

# This is the commit message #29:

delete the doc directly, handle returns

# This is the commit message #30:

refactor a bit

# This is the commit message #31:

remove unreachable clause

# This is the commit message #32:

update spec

# This is the commit message #33:

expose more paths

# This is the commit message #34:

expose more paths

# This is the commit message #35:

fix specs

# This is the commit message #36:

match on proper return

# This is the commit message #37:

update spec

# This is the commit message #38:

update account docs

# This is the commit message #39:

should save the docs

# This is the commit message #40:

add missing types

# This is the commit message #41:

hackney wants a binary

# This is the commit message #42:

hackney wants a binary

# This is the commit message #43:

more strict checking of design docs

# This is the commit message #44:

update type

# This is the commit message #45:

update usages

# This is the commit message #46:

add announcement for 4.3

# This is the commit message #47:

update spelling

# This is the commit message #48:

fix spelling

# This is the commit message #49:

ref the local fun

# This is the commit message #50:

save the accounts version if not_found

# This is the commit message #51:

return the account db version

# This is the commit message #52:

hide $ from formatter

# This is the commit message #53:

make sure to update the account doc

# This is the commit message #54:

move saving account docs to updates instaed

# This is the commit message #55:

right function name this time
sudomabider pushed a commit to OpenTelecom/kazoo that referenced this pull request Oct 16, 2018
@BorigTheDwarf
Merged in KAZ-355 (pull request 2600hz#38)
50ebc4f 
[KAZ-355] Update presence state when DND toggled
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jamesaimonetti @k-anderson