Skip to content

When using the Usage API the uri field in response contains /todo.jsp #1689

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.

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

Closed
deruelle opened this issue Jan 4, 2017 · 4 comments
Closed

Comments

@deruelle
Copy link
Member

deruelle commented Jan 4, 2017

See

True for both Daily and Monthly

curl --user 'ACXXX:YYYYY' 'https://cloud.restcomm.com/restcomm/2012-04-24/Accounts/ACXXXX/Usage/Records/Monthly.json?EndDate=2016-12-31&StartDate=2016-12-01&Category=CALLS'
[
{
"category": "calls",
"description": "Total Calls",
"account_sid": "ACXXX",
"start_date": "2016-12-02",
"end_date": "2016-12-31",
"usage": "12402",
"usage_unit": "minutes",
"count": "638",
"count_unit": "calls",
"price": "0",
"price_unit": "USD",
"uri": "/todo.json"
}

@deruelle deruelle added this to the 8.1.0 milestone Jan 4, 2017
@abdulazizali77
Copy link
Contributor

abdulazizali77 commented Jan 4, 2017

Hello, had a quick look, seems to be from here
https://github.com/RestComm/Restcomm-Connect/blob/master/restcomm/restcomm.application/src/main/webapp/WEB-INF/sql/usage.xml
potentially have to modify the uri in getUsageCalls toUsageRecord after the SELECT, or pass the uri in then fetch it out again :D
I reckon #1690 is due to the mybatis functions, might have to test the CONVERT(MIN())
Im off the mark about #1690 investigating precision loss between DateTime to Date to DateTime conversion
I dont mind taking these two issues up!

@deruelle
Copy link
Member Author

deruelle commented Jan 5, 2017

@abdulazizali77 sounds great, please by all means do take those issues !
Can you accept the invitation at https://github.com/RestComm/Restcomm-Connect/invitations and sign the Contributor License Agreement at https://telestax.com/open-source/#Contribute to I can assign those issues to you ?

@abdulazizali77
Copy link
Contributor

Done! Ill followup with PRs by the end of the week! (hopefully)

@deruelle
Copy link
Member Author

deruelle commented Jan 6, 2017

@abdulazizali77 awesome. Assigned to you !

abdulazizali77 added a commit to abdulazizali77/Restcomm-Connect that referenced this issue Apr 1, 2017
… in Uri. Pass in Uri and rewrite Uri in sql query
abdulazizali77 added a commit to abdulazizali77/Restcomm-Connect that referenced this issue Apr 1, 2017
abdulazizali77 added a commit to abdulazizali77/Restcomm-Connect that referenced this issue Apr 1, 2017
abdulazizali77 added a commit to abdulazizali77/Restcomm-Connect that referenced this issue Apr 1, 2017
abdulazizali77 added a commit to abdulazizali77/Restcomm-Connect that referenced this issue Apr 1, 2017
abdulazizali77 added a commit to abdulazizali77/Restcomm-Connect that referenced this issue Apr 1, 2017
abdulazizali77 added a commit to abdulazizali77/Restcomm-Connect that referenced this issue Apr 1, 2017
abdulazizali77 added a commit to abdulazizali77/Restcomm-Connect that referenced this issue Apr 1, 2017
abdulazizali77 added a commit to abdulazizali77/Restcomm-Connect that referenced this issue Apr 1, 2017
abdulazizali77 added a commit to abdulazizali77/Restcomm-Connect that referenced this issue Apr 1, 2017
@otsakir otsakir closed this as completed Apr 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants