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

BLUE-150 Created config for the number of cycles to keep shard calculations data #46

Merged
merged 20 commits into from
Aug 4, 2024

Conversation

jairajdev
Copy link
Contributor

No description provided.

Copy link

linear bot commented Jul 2, 2024

@@ -98,13 +98,13 @@
})
promises.push(promise)
} catch (e) {
Logger.mainLogger.error('Error', e)
Logger.mainLogger.error(`Gossip Error to archiver ${archiver.ip}: ${archiver.port}`, e)

Check warning

Code scanning / CodeQL

Log injection Medium

Log entry depends on a
user-provided value
.
src/Data/Collector.ts Fixed Show fixed Hide fixed
const currentCycle = getCurrentCycleCounter()
if (currentCycle - cycle > 2) {
Logger.mainLogger.error(
`Found receipt with cycle older than 2 cycles ${txId}, ${cycle}, ${timestamp}, ${currentCycle}`

Check warning

Code scanning / CodeQL

Log injection Medium

Log entry depends on a
user-provided value
.
unsubscribeDataSender(sender.nodeInfo.publicKey)
return
}
Logger.mainLogger.debug(`RECEIVED ACCOUNTS DATA FROM ${sender.nodeInfo.ip}:${sender.nodeInfo.port}`)

Check warning

Code scanning / CodeQL

Log injection Medium

Log entry depends on a
user-provided value
.
if (getCurrentCycleCounter() > GENESIS_ACCOUNTS_CYCLE_RANGE.endCycle) {
Logger.mainLogger.error(
'Account data is not meant to be received after the genesis accounts cycle range',
getCurrentCycleCounter()

Check warning

Code scanning / CodeQL

Log injection Medium

Log entry depends on a
user-provided value
.
@mhanson-github mhanson-github merged commit dddd88a into dev Aug 4, 2024
7 checks passed
@mhanson-github mhanson-github deleted the cycle-shard-data-config branch September 2, 2024 04:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants