11 Commits (mr26.1)

Author	SHA1	Message	Date
Kirill Solomko	7063648062	MT#65432 add cf sets subscriber_id support subscriberadmin ... * subscriberadmin role can now correctly provide with a subscriber_id under which the cf set is created/updated, only ones that belong to the same customer are valid. The behaviour remains unchanged if the subscriber_id is omitted as the subscriberadmin id is used in this case. Change-Id: I51f0333e51865e91a127338ac5f888eb88d15271 (cherry picked from commit 9470652038) (cherry picked from commit 3401540d0a)	2 weeks ago
Kirill Solomko	ce12aed828	MT#64923 callforward sets add own boolean field ... * /api/callforwards (per set), /api/cfdestinationsets, /api/cfsourcesets, /api/cfbnumbersets, /api/cftimesets return "own" field for subscriber and subscriberadmin roles only, this field indicates if the call forward set belongs to the same subscriber that requests it. * /api/callforwards fix default value for 'cfr' and 'cfo' cftypes from null to {}, to match all the other cftype default values. Change-Id: I9c5c60dfb16301703341c1ba17c751079f79ae8b	3 months ago
Kirill Solomko	205b27a267	MT#59478 API better transaction and error handling ... * Role/Entities*: POST/PUT/PATCH/DELETE methods changes: - support deadlock detection and transaction retry (2 retry attempts at the moment) - improve transaction control, use local $guard instead of saving the ref to $c->stash, as in that case it went out of scope too late and also reported an error message into the log about abnormal $guard out of scope interruption - move all non transaction related code outside of the scope - add error handling when methods such as update_item, and a like do not return the expected data, instead of simply going out of scope and resulting in an uncontrolled reply * Role/API: - rework transaction control: + get_transaction_control() is renamed to start_transaction() to better reflect what it does + complete_transaction() is renamed to commit_transaction() + remove unused %params arg + pass $guard into commit_transaction() instead of having it stored as $c->stash->{transaction_guard) that caused the $guard ref to be destroyed much late than expected (there was also a typo as transaction_quard, which is not relevant anymore with the changes + add check_deadlock() that is invoked when an exception is caught or an $c->errors contain an error, and if the error message represents a transaction error, the transaction block is re-invoked via "goto TX_START" - rework error(): + it now accepts args as following: ($self, $c, $code, $message, @errors) # code -> returned as HTTP code in the reply # message -> returned as HTTP message in the reply # errors -> contain errors for internal logging, last element often contains a DBIx exception + populates all @errors into $c->error so they are available on demend in the code via $c->error or $c->last_error + $c->log->error is not invoked now as the errors become printed in log_response() - log_response() now prints collected errors from $c->error correctly as a separate log line, that is alike to the other api logs so that those can be looked up by the request's tx_id, also all errors are now printed only into api.log * Adjust all $self->error() calls in catch($e) to include $e as the last argument, as well as the duplicate $c->log->error is removed from those ocassions * Remove all $c->log->error() calls as they are replaced with either $self->error() (that logs it correctly into api.log) or $c->error('err') that also adds it correctly into api.log * API::CallForwards: rework to use Entities/EntitiesItem * API::Contracts: rework POST to use Entities * API::PeeringGroups: rework POST to use Entities * API::SubscriberRegistrations: rework POST to use Entities * API::RewriteRuleSets: improve create_item() functionality * Utils/Message: add 'api_retry' log type * $c->session->{api_request_tx_id} is changed to $c->stash->{api_request_tx_id} because sometimes the session ref is different and a different tx_id becomes used Change-Id: I633ce7a8047b1bf00a2f6889003088edf0825dcd	2 years ago
Kirill Solomko	dee6b60c6e	MT#56208 additional inherited cfs improvements for subscribers ... * API: GET collection returns only distinct cf sets * UI: inherited cf sets are now shown in the advanced view and having suffix '(inherited)' to distinguish them from the own ones as they are not available in the "Manage Sets" edit dialog. Change-Id: Ifd8c1a46fcb4c2c66f8f63268a1e91b80792c216	4 years ago
Kirill Solomko	e90ba9d517	MT#56208 allow subscriber role access to cf sets ... * subscriber role can now GET call forward sets that does not belong to the user but used in call forwards assigned to the user * the user can only GET the sets but cannot update/delete them (403 Forbidden is raised in case of attempt) Change-Id: I391b802d962f4bbeae991e3046ac3f132a19edb6	4 years ago
Kirill Solomko	4c1a9d9c11	MT#56029 /api/cfbnumbersets subscriberadmin role change ... * subscriberadmin is now able to fetch cfbnumbersets of other subsribers that belong to the customer Change-Id: Ib92c161f320257ba2d6614e10d9eaf03247e7014	4 years ago
Kirill Solomko	ce664263b2	TT#65101 add ccareadmin ccare roles ... * ccareadmin and ccare roles have full access to Customers, Subscribers and their preferences/settings, and read-only access to BillingProfiles,InvoceTemplates, EmailTemplates * ccare role is restricted to the related reseller Change-Id: I6cf7d3adf912f0fa98d1ef5c02abea2f4331ec4b	7 years ago
Irina Peshinskaya	c76d300f7d	TT#40503 Remove explicit role checking for journal links ... Change-Id: Id33dc65590d87f15fa565d4a400bd059f6bd9654	8 years ago
Irina Peshinskaya	9ed0657d4c	TT#40503 Show journal links to allowed roles only ... Change-Id: I547b2db4767b1e383506bc812b6486db944fceec	8 years ago
Gerhard Jungwirth	9210ac0487	TT#37206 is_regexp flag for callforwardings ... implemented for Source Sets and B-Number Set in: * Panel * /api/cfbnumbersets/ * /api/cfsourcesets/ * /api/callforwards/ in addition, the fields have also been documented in the forms (and therefore also in the API documentation) Change-Id: I955dd9addf1e61ec7516b4c518adec151248bacc	8 years ago
Gerhard Jungwirth	f5687b2294	TT#37205 bnumber cf in API: complex /api/cf* ... * /api/cfmappings * /api/cfbnumbersets small fix: "anonymous" doesn't make sense for b-numbers Change-Id: I13fe97bb35c7c34f8919932391dea86bd492a291	8 years ago