Skip to content
GitLab
Explore
Sign in
Register
Primary navigation
Search or go to…
Project
H
hive-js
Manage
Activity
Members
Labels
Plan
Issues
Issue boards
Milestones
Wiki
Code
Merge requests
Repository
Branches
Commits
Tags
Repository graph
Compare revisions
Snippets
Deploy
Releases
Container Registry
Model registry
Monitor
Incidents
Analyze
Value stream analytics
Contributor analytics
Repository analytics
Model experiments
Help
Help
Support
GitLab documentation
Compare GitLab plans
Community forum
Contribute to GitLab
Provide feedback
Keyboard shortcuts
?
Snippets
Groups
Projects
Show more breadcrumbs
hive
hive-js
Commits
29470585
Commit
29470585
authored
4 years ago
by
Mahdi Yari
Browse files
Options
Downloads
Patches
Plain Diff
remove the bad practice
parent
80e96819
No related branches found
No related tags found
1 merge request
!20
Fix account history
Changes
1
Hide whitespace changes
Inline
Side-by-side
Showing
1 changed file
src/api/transports/http.js
+10
-10
10 additions, 10 deletions
src/api/transports/http.js
with
10 additions
and
10 deletions
src/api/transports/http.js
+
10
−
10
View file @
29470585
...
...
@@ -60,16 +60,16 @@ export default class HttpTransport extends Transport {
const
id
=
data
.
id
||
this
.
id
++
;
let
params
=
[
api
,
data
.
method
,
data
.
params
];
//SPECIAL CODE - can be removed after all API node operators upgrade to get the updated get_account_history api call
if
(
this
.
options
.
uri
!==
'
https://api.hive.blog
'
&&
data
.
method
===
'
get_account_history
'
&&
data
.
params
.
length
>=
4
)
{
//We are experimenting with a new version of get_account_history that can now take up to 5 params
//but this is only deployed on api.hive.blog nodes, so if this particular request is going to a different
//backend, just strip the extra parameters off the call to avoid breaking it. Once all API nodes have upgraded
//this code can be removed.
while
(
data
.
params
.
length
>
3
)
data
.
params
.
pop
();
params
=
[
api
,
data
.
method
,
data
.
params
];
}
//
if (this.options.uri !== 'https://api.hive.blog' && data.method === 'get_account_history' && data.params.length >= 4)
//
{
//
//We are experimenting with a new version of get_account_history that can now take up to 5 params
//
//but this is only deployed on api.hive.blog nodes, so if this particular request is going to a different
//
//backend, just strip the extra parameters off the call to avoid breaking it. Once all API nodes have upgraded
//
//this code can be removed.
//
while (data.params.length > 3)
//
data.params.pop();
//
params = [api, data.method, data.params];
//
}
//END SPECIAL CODE
const
retriable
=
this
.
retriable
(
api
,
data
);
const
fetchMethod
=
this
.
options
.
fetchMethod
;
...
...
This diff is collapsed.
Click to expand it.
Preview
0%
Loading
Try again
or
attach a new file
.
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Save comment
Cancel
Please
register
or
sign in
to comment