Skip to content

Include the git blob id of the dir-index bundle in the ETag#7360

Merged
Stebalien merged 2 commits intomasterfrom
feat/better_dir_etag
May 25, 2020
Merged

Include the git blob id of the dir-index bundle in the ETag#7360
Stebalien merged 2 commits intomasterfrom
feat/better_dir_etag

Conversation

@ribasushi
Copy link
Contributor

While the content of raw files retrieved via the gateway should never
change, the look and feel of the directory index can and will change
between versions of go-ipfs.

Incorporate the hash of assets/bindata.go into the ETag when appropriate

@Stebalien this is what we spoke about last year @ camp. Goes with @jessicaschilling's recent changes.

While the content of raw files retrieved via the gateway should never
change, the look and feel of the directory index can and will change
between versions of go-ipfs.

Incorporate the hash of assets/bindata.go into the ETag when appropriate
@ribasushi ribasushi requested review from Stebalien and lidel May 25, 2020 18:47
@ribasushi
Copy link
Contributor Author

Context: #6465 (review)

@@ -0,0 +1,5 @@
package assets
Copy link
Member

Choose a reason for hiding this comment

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

Can we add a comment here to say that this shouldn't be edited?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

✅ done

@Stebalien Stebalien merged commit 707b550 into master May 25, 2020
@Stebalien
Copy link
Member

Thanks!

@Stebalien Stebalien mentioned this pull request May 26, 2020
77 tasks
@hacdias hacdias deleted the feat/better_dir_etag branch May 9, 2023 10:59
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.

2 participants