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.

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

on submitNonce there is no deadline #40

Closed
matbogdan79 opened this issue Jul 29, 2017 · 5 comments
Closed

on submitNonce there is no deadline #40

matbogdan79 opened this issue Jul 29, 2017 · 5 comments
Assignees
Milestone

Comments

@matbogdan79
Copy link

Please check the submitNonce procedure for the pool.
There is no deadline or x-deadline that should get the current miner nonce calculated deadline.
The x-deadline is used by CreepSky to pass the actual deadline in sec or dedline is used by Blago to pass the deadline based on baseTarget.

on NetworkSubmitPoolNonceTask.java is only one change:

  ContentResponse response = httpClient.POST(poolServer + "/burst")
    .param("requestType", "submitNonce")
    .param("accountId", numericAccountId)
    .param("nonce", nonce.toString())
   .param("x-deadline", calculatedDeadline.toString())
    .header("X-Miner", HEADER_MINER_NAME)
    .header("X-Capacity", String.valueOf(gb))
    .timeout(connectionTimeout, TimeUnit.MILLISECONDS)
    .send();

Please add it to your miner.

@de-luxe
Copy link
Owner

de-luxe commented Jul 29, 2017

@matbogdan79 Doesn't the pool have to calculate/validate the deadline anyway by nonce? Sure i can add that, but i don't get why it is needed. Please point me to the pool software repository using that.

@matbogdan79
Copy link
Author

matbogdan79 commented Jul 29, 2017

This is used by uray pool source.
image
I am trying to prevent with a pool checker this kind of submit by jminer - this is the only miner that have this missing.
This is required to prevent bad nonces to be processed by the java wallet and therefore reduce the CPU time on that.
Blago miner has it , CreepMiner has it.
Please add it to jminer - this being used by the most big miners and we need a way to filter the bad nonce's.
The uray pool calculate/validate it trough the java wallet and that is a pain when miners submit nonce's like that.

Thank you for understanding and cooperation.

@de-luxe
Copy link
Owner

de-luxe commented Jul 29, 2017

@matbogdan79 Thanks for the hint, i will add it.

@de-luxe de-luxe self-assigned this Jul 29, 2017
@de-luxe de-luxe added this to the 0.4.x-RELEASE milestone Jul 29, 2017
@Creepsky
Copy link

Creepsky commented Jul 29, 2017

Just to make this clear because I already had this problem when I implemented this :)
Blago miner and creepminer uses this field mainly intern for forwarding.

Blago uses /burst?requestType=submitNonce&...&deadline=123, where deadline is the raw deadline (dont know how to call this, its basically real deadline * basetarget.

creepminer uses X-Deadline in the POST body of the submit, what represents the deadline in seconds as a string.

Maybe we should standardize the fields so the miners will become all compatible :) and also for @matbogdan79 it will be easier to check.

@de-luxe
Copy link
Owner

de-luxe commented Jul 30, 2017

@Creepsky Ok with me, i'm also open to in general discuss miner<->pool api enhancements ... i would have added X-Deadline to the header like X-Miner and X-Capacity ...

de-luxe added a commit that referenced this issue Mar 8, 2018
- #40 on submitNonce there is no deadline
- #44 Compatibility for chaining JMiner to CreepMiner
- #45 Failed to parse response from server.
- show stop drive info only on debug if showDriveInfo enabled
- info message on try run with java9
- some cleanup and optimize, update to latest lib versions
@de-luxe de-luxe closed this as completed Mar 8, 2018
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