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.

Sign up for GitHub

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

Jump to bottom

Improve mist error handling if import of JS file fails #279

Closed
Gustav-Simonsson opened this issue Jan 30, 2015 · 0 comments
Closed

Improve mist error handling if import of JS file fails #279

Gustav-Simonsson opened this issue Jan 30, 2015 · 0 comments
Labels
type:feature

Comments

@Gustav-Simonsson
Copy link

If a non-JS file is imported in Mist, it can cause Mist to hang.

From bug report:

"It never checks if it is a js file. When it is run in the Otto VM, it is not checking for errors as well. It should either check before trying evaluate it or when running it in the otto vm?

https://github.com/ethereum/go-ethereum/blob/develop/javascript/javascript_runtime.go#L34
@fjl fjl closed this as completed Jun 23, 2015
nonsense added a commit that referenced this issue Mar 23, 2018
@nonsense
Merge pull request #279 from ethersphere/rename-dbstore-to-LDBStore
a2ff1aa 
rename DbStore to LDBStore
prestonvanloon added a commit to prestonvanloon/go-ethereum that referenced this issue Jul 15, 2018
@prestonvanloon
update rules_go and gazelle (ethereum#279)
2777b5e
ngtuna added a commit to ngtuna/tomochain that referenced this issue Nov 16, 2018
@ngtuna
Merge pull request ethereum#279 from ngtuna/updateM1-failure
b757de8 
stop node when updateM1 fails
Zergity added a commit to Zergity/go-ethereum that referenced this issue Apr 28, 2020
@Zergity
Merge pull request ethereum#279 from nextyio/reverted-log
4d0740a 
Receipt log for reverted tx
@snyk-bot snyk-bot mentioned this issue Jul 16, 2020
Open
AusIV pushed a commit to NoteGio/go-ethereum that referenced this issue Feb 22, 2021
@meowsbits
cmd/faucet/main: replace faucet node datadir s/MultiFaucet/CoreFaucet/ (
d9dacfb 
ethereum#279)

Use datadir migration logic to rename directory if exists.
maoueh pushed a commit to streamingfast/go-ethereum that referenced this issue Aug 13, 2021
@tynes
scripts: add ws (ethereum#279)
b54a387
tony-ricciardi pushed a commit to tony-ricciardi/go-ethereum that referenced this issue Jan 20, 2022
@asaj
Batch insertion of headers in ultralight sync mode (ethereum#279)
bed6cf6
maoueh pushed a commit to streamingfast/go-ethereum that referenced this issue May 3, 2022
@ferranbt
Use stream to send debug files (ethereum#279)
4829114 
* Fix protobuf file

* Small fix

* Use grpc streams to send big debug files
@snyk-bot snyk-bot mentioned this issue Aug 23, 2022
Open
weiihann pushed a commit to weiihann/go-ethereum that referenced this issue Nov 30, 2023
@gballet
port kaustinen code on top of shapella-rebased branch
ba2fa09 
activate proof generation on fork  + remove code dups

use go-verkle's post-state API to verify proofs (ethereum#262)

use prague as the verkle activation fork (ethereum#263)

upgrade to latest go-ipa

activate verkle transition in "miner" (ethereum#265)

fix: do not force cancunTime upon verkle activation

workaround: do not use root translation in replay

workaround: deactivate overlay transition for now

fixes from trying to get the devnet to work (ethereum#267)

this line was left out from the previous commit

upgrade to go-verkle with fixed newvalue serialization

fix: ensure point cache isn't nil in copy (ethereum#268)

fix: dependency cycle in tests (ethereum#269)

upgrade to latest go-verkle

fix: write trie preimage data to db (ethereum#274)

fix: zero-root in produced block + sync (ethereum#275)

upgrade go-ipa

fix build

fix typo

include review feedback

add switch to add proofs to blocks (ethereum#278)

add fee recipient to witness (ethereum#279)

touch all fields in withdrawal account header (ethereum#277)
sduchesneau pushed a commit to streamingfast/go-ethereum that referenced this issue Feb 26, 2024
@PlasmaPower
Merge pull request ethereum#279 from OffchainLabs/merge-1.13
6eb9331 
Merge 1.13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type:feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@fjl @Gustav-Simonsson @zelig