[DEV] Added OAuth call to MPesa and Account Balance Query #85
Conversation
[MERGE] With openMF
| exchange.setProperty("tokenResponseCode", "200"); | ||
| exchange.setProperty("access_token", response.getAccess_token()); | ||
| } else { | ||
| exchange.setProperty("tokenResponseCode", String.valueOf(responseEntity.getStatusCode().value())); |
There was a problem hiding this comment.
Is there an error response JSON?
There was a problem hiding this comment.
Yeah, there are error codes in Response JSON.
{
"requestId": "18641-15432775-1",
"errorCode": "400.008.01",
"errorMessage": "Invalid Authentication passed"
}
work/mpesa/application.yml
Outdated
| - name: tn04 | ||
| port: 48889 | ||
| - name: quotes #post notification | ||
| base: interoperation/quotes |
work/mpesa/application.yml
Outdated
| - name: parties | ||
| base: fineract-provider/api/v1/interoperation/parties | ||
| - name: quotes | ||
| base: fineract-provider/api/v1/interoperation/quotes |
work/mpesa/application.yml
Outdated
| - name: requests | ||
| base: fineract-provider/api/v1/interoperation/requests | ||
| - name: parties | ||
| base: fineract-provider/api/v1/interoperation/parties |
There was a problem hiding this comment.
Please evaluate if this is required or not.
work/mpesa/application.yml
Outdated
| base: mpesa/accountbalance/v1/query | ||
| bindings: #ott -> hub | ||
| - name: binding-basic-settings | ||
| host: http://0.0.0.0 |
There was a problem hiding this comment.
Is there IP whitelisting on Safaricom side? If this has to be a callback and you need an IP independent of Mifos / Fineract SSL, we can provide you IP of the reverse proxy server we have.
There was a problem hiding this comment.
Whitelisting is done before every transaction through Request URL, as mentioned in my proposal. So, for C2B payments, we'll be calling an API to register our IPs for callbacks and then call APIs for the transaction.
| ; | ||
|
|
||
| from("direct:commitTransaction") | ||
| .id("commitTransaction") |
There was a problem hiding this comment.
What is the purpose of this in this particular pull request?
There was a problem hiding this comment.
Change the naming if this caters to balance check.
This PR consists of:
TODO: