Skip to content

Conversation

@cooganb
Copy link
Contributor

@cooganb cooganb commented Jan 23, 2018

This PR adds a base64 data blob in the head of the generic HTML templates used by the HTTP Swarm client.

This PR has been approved by 2 members of the Swarm development team in ethersphere#208.

It is a contribution to the goal of improving the http interface for Swarm users. Please let me know if you have any questions.

Copy link
Member

@karalabe karalabe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SGTM

@karalabe karalabe added this to the 1.8.0 milestone Jan 23, 2018
@cooganb
Copy link
Contributor Author

cooganb commented Jan 23, 2018

@karalabe Thanks so much! I so appreciate it.

I also have another PR waiting for review also related to http interface of Swarm, ethereum#15926...would you mind taking a look at that?

Thanks again!

Copy link
Member

@karalabe karalabe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, the swarm tests fail on this one. Please fix those.

@cooganb
Copy link
Contributor Author

cooganb commented Jan 24, 2018

@karalabe will do! on it now

@karalabe
Copy link
Member

karalabe commented Feb 5, 2018

Ping?

@cooganb
Copy link
Contributor Author

cooganb commented Feb 5, 2018

hey @karalabe ! Thanks for following up. Swarm Dev incorrectly stated it was an error of whisper protocol (which is true for one test shown below) but another test fails in server_test.go, which I just realized while typing this to you. Working on it now!

FAIL coverage: 59.3% of statements FAIL github.com/ethereum/go-ethereum/whisper/whisperv6 23.232s util.go:45: exit status 1 exit status 1

@karalabe
Copy link
Member

karalabe commented Feb 6, 2018

Tests are still failing.

@cooganb
Copy link
Contributor Author

cooganb commented Feb 6, 2018 via email

@GitCop
Copy link

GitCop commented Feb 12, 2018

Thank you for your contribution! Your commits seem to not adhere to the repository coding standards

  • Commit: 4af6c8c1ab14527bc6a27c7c933b8cc2c12b1e14
  • Commits must be prefixed with the package(s) they modify

Please check the contribution guidelines for more details.


This message was auto-generated by https://gitcop.com

@cooganb
Copy link
Contributor Author

cooganb commented Feb 13, 2018

@karalabe main tests are fixed, rebasing from master right now to correct one more.

@cooganb
Copy link
Contributor Author

cooganb commented Feb 13, 2018

@karalabe all the swarm tests are passing -- the fail test is coming from

go build _/var/folders/bb/n7t3rs157850byt_jfdcq9k80000gn/T/gomobile-work-757186050/src/gomobile_bind: invalid flag in #cgo CFLAGS: -fmodules
/Users/travis/gopath/src/github.com/ethereum/go-ethereum/build/bin/gomobile: darwin-armv7: go build -pkgdir=/Users/travis/gopath/pkg/gomobile/pkg_darwin_arm -tags ios ios -v -buildmode=c-archive -o /var/folders/bb/n7t3rs157850byt_jfdcq9k80000gn/T/gomobile-work-757186050/geth-arm.a /var/folders/bb/n7t3rs157850byt_jfdcq9k80000gn/T/gomobile-work-757186050/src/iosbin/main.go failed: exit status 1
util.go:45: exit status 1
exit status 1
The command "go run build/ci.go xcode -signer IOS_SIGNING_KEY -deploy trunk -upload gethstore/builds" exited with 1.

@karalabe karalabe modified the milestones: 1.8.0, 1.8.1 Feb 13, 2018
Copy link
Member

@karalabe karalabe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@karalabe karalabe merged commit 4e61ed0 into ethereum:master Feb 15, 2018
@cooganb
Copy link
Contributor Author

cooganb commented Feb 15, 2018

@karalabe thank you!!

prestonvanloon pushed a commit to prestonvanloon/go-ethereum that referenced this pull request Apr 2, 2018
)

* swarm: added script to HTML header to create favicon addresses ethereum#153

* swarm: moved data blob direclty into link tag, removed script

* swarm: added favicon info to other html templates

* swarm: fixing test errors

* swarm: fixing favicon test

* swarm: fixing travis tests

* swarm: added script to HTML header to create favicon addresses ethereum#153

* swarm: moved data blob direclty into link tag, removed script

* swarm: added favicon info to other html templates

* swarm: fixing test errors

* swarm: fixing favicon test

* swarm: fixing travis tests
mariameda pushed a commit to NiluPlatform/go-nilu that referenced this pull request Aug 23, 2018
)

* swarm: added script to HTML header to create favicon addresses ethereum#153

* swarm: moved data blob direclty into link tag, removed script

* swarm: added favicon info to other html templates

* swarm: fixing test errors

* swarm: fixing favicon test

* swarm: fixing travis tests

* swarm: added script to HTML header to create favicon addresses ethereum#153

* swarm: moved data blob direclty into link tag, removed script

* swarm: added favicon info to other html templates

* swarm: fixing test errors

* swarm: fixing favicon test

* swarm: fixing travis tests
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.

5 participants