Re: [tor-bugs] #31632 [Core Tor/Tor]: hs-v3: Service doesn't re-upload descriptor on circuit failure

[Tor Bug Tracker & Wiki]

#31632: hs-v3: Service doesn't re-upload descriptor on circuit failure
-+-
 Reporter:  dgoulet  |  Owner:  dgoulet
 Type:  defect   | Status:
 |  needs_revision
 Priority:  Medium   |  Milestone:  Tor:
 |  0.4.4.x-final
Component:  Core Tor/Tor |Version:
 Severity:  Normal   | Resolution:
 Keywords:  tor-hs, 042-deferred-20190918,   |  Actual Points:
  043-deferred   |
Parent ID:   | Points:  0.5
 Reviewer:  asn  |Sponsor:
 |  Sponsor27-can
-+-
Changes (by dgoulet):

 * sponsor:  Sponsor27-must => Sponsor27-can
 * parent:  #30200 =>


Comment:

 Unparenting so we can close the parents. This will get merged into 044 at
 some point.

--
Ticket URL: 
Tor Bug Tracker & Wiki 
The Tor Project: anonymity online
___
tor-bugs mailing list
tor-bugs@lists.torproject.org
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs

Re: [tor-bugs] #31632 [Core Tor/Tor]: hs-v3: Service doesn't re-upload descriptor on circuit failure

[Tor Bug Tracker & Wiki]

#31632: hs-v3: Service doesn't re-upload descriptor on circuit failure
---+---
 Reporter:  dgoulet|  Owner:  dgoulet
 Type:  defect | Status:
   |  needs_revision
 Priority:  Medium |  Milestone:  Tor:
   |  0.4.3.x-final
Component:  Core Tor/Tor   |Version:
 Severity:  Normal | Resolution:
 Keywords:  tor-hs, 042-deferred-20190918  |  Actual Points:
Parent ID:  #30200 | Points:  0.5
 Reviewer:  asn|Sponsor:
   |  Sponsor27-must
---+---
Changes (by asn):

 * status:  needs_review => needs_revision


Comment:

 Replying to [comment:11 dgoulet]:
 > Some discussions on the PR that I need asn to go through.

 Replied but probably failed to provide useful answers :/

--
Ticket URL: 
Tor Bug Tracker & Wiki 
The Tor Project: anonymity online
___
tor-bugs mailing list
tor-bugs@lists.torproject.org
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs

Re: [tor-bugs] #31632 [Core Tor/Tor]: hs-v3: Service doesn't re-upload descriptor on circuit failure

[Tor Bug Tracker & Wiki]

#31632: hs-v3: Service doesn't re-upload descriptor on circuit failure
---+---
 Reporter:  dgoulet|  Owner:  dgoulet
 Type:  defect | Status:  needs_review
 Priority:  Medium |  Milestone:  Tor:
   |  0.4.3.x-final
Component:  Core Tor/Tor   |Version:
 Severity:  Normal | Resolution:
 Keywords:  tor-hs, 042-deferred-20190918  |  Actual Points:
Parent ID:  #30200 | Points:  0.5
 Reviewer:  asn|Sponsor:
   |  Sponsor27-must
---+---
Changes (by dgoulet):

 * status:  needs_revision => needs_review


Comment:

 Some discussions on the PR that I need asn to go through.

--
Ticket URL: 
Tor Bug Tracker & Wiki 
The Tor Project: anonymity online
___
tor-bugs mailing list
tor-bugs@lists.torproject.org
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs

Re: [tor-bugs] #31632 [Core Tor/Tor]: hs-v3: Service doesn't re-upload descriptor on circuit failure

[Tor Bug Tracker & Wiki]

#31632: hs-v3: Service doesn't re-upload descriptor on circuit failure
---+---
 Reporter:  dgoulet|  Owner:  dgoulet
 Type:  defect | Status:
   |  needs_revision
 Priority:  Medium |  Milestone:  Tor:
   |  0.4.3.x-final
Component:  Core Tor/Tor   |Version:
 Severity:  Normal | Resolution:
 Keywords:  tor-hs, 042-deferred-20190918  |  Actual Points:
Parent ID:  #30200 | Points:  0.5
 Reviewer:  asn|Sponsor:
   |  Sponsor27-must
---+---
Changes (by asn):

 * status:  needs_review => needs_revision


Comment:

 Good stuff! A review has been done!

--
Ticket URL: 
Tor Bug Tracker & Wiki 
The Tor Project: anonymity online
___
tor-bugs mailing list
tor-bugs@lists.torproject.org
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs

Re: [tor-bugs] #31632 [Core Tor/Tor]: hs-v3: Service doesn't re-upload descriptor on circuit failure

[Tor Bug Tracker & Wiki]

#31632: hs-v3: Service doesn't re-upload descriptor on circuit failure
---+---
 Reporter:  dgoulet|  Owner:  dgoulet
 Type:  defect | Status:  needs_review
 Priority:  Medium |  Milestone:  Tor:
   |  0.4.3.x-final
Component:  Core Tor/Tor   |Version:
 Severity:  Normal | Resolution:
 Keywords:  tor-hs, 042-deferred-20190918  |  Actual Points:
Parent ID:  #30200 | Points:  0.5
 Reviewer:  asn|Sponsor:
   |  Sponsor27-must
---+---
Changes (by dgoulet):

 * status:  assigned => needs_review


Comment:

 Branch: `ticket31632_043_01`
 PR: https://github.com/torproject/tor/pull/1593

--
Ticket URL: 
Tor Bug Tracker & Wiki 
The Tor Project: anonymity online
___
tor-bugs mailing list
tor-bugs@lists.torproject.org
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs

Re: [tor-bugs] #31632 [Core Tor/Tor]: hs-v3: Service doesn't re-upload descriptor on circuit failure

[Tor Bug Tracker & Wiki]

#31632: hs-v3: Service doesn't re-upload descriptor on circuit failure
---+---
 Reporter:  dgoulet|  Owner:  dgoulet
 Type:  defect | Status:  assigned
 Priority:  Medium |  Milestone:  Tor:
   |  0.4.3.x-final
Component:  Core Tor/Tor   |Version:
 Severity:  Normal | Resolution:
 Keywords:  tor-hs, 042-deferred-20190918  |  Actual Points:
Parent ID:  #30200 | Points:  0.5
 Reviewer:  asn|Sponsor:
   |  Sponsor27-must
---+---

Comment (by dgoulet):

 Need for #32020 to be merged upstream before I can fix this.

--
Ticket URL: 
Tor Bug Tracker & Wiki 
The Tor Project: anonymity online
___
tor-bugs mailing list
tor-bugs@lists.torproject.org
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs

Re: [tor-bugs] #31632 [Core Tor/Tor]: hs-v3: Service doesn't re-upload descriptor on circuit failure

[Tor Bug Tracker & Wiki]

#31632: hs-v3: Service doesn't re-upload descriptor on circuit failure
---+---
 Reporter:  dgoulet|  Owner:  dgoulet
 Type:  defect | Status:  assigned
 Priority:  Medium |  Milestone:  Tor:
   |  0.4.3.x-final
Component:  Core Tor/Tor   |Version:
 Severity:  Normal | Resolution:
 Keywords:  tor-hs, 042-deferred-20190918  |  Actual Points:
Parent ID:  #30200 | Points:  0.5
 Reviewer:  asn|Sponsor:
   |  Sponsor27-must
---+---
Changes (by dgoulet):

 * status:  new => assigned
 * owner:  (none) => dgoulet


--
Ticket URL: 
Tor Bug Tracker & Wiki 
The Tor Project: anonymity online
___
tor-bugs mailing list
tor-bugs@lists.torproject.org
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs

Re: [tor-bugs] #31632 [Core Tor/Tor]: hs-v3: Service doesn't re-upload descriptor on circuit failure

[Tor Bug Tracker & Wiki]

#31632: hs-v3: Service doesn't re-upload descriptor on circuit failure
---+---
 Reporter:  dgoulet|  Owner:  (none)
 Type:  defect | Status:  new
 Priority:  Medium |  Milestone:  Tor:
   |  0.4.3.x-final
Component:  Core Tor/Tor   |Version:
 Severity:  Normal | Resolution:
 Keywords:  tor-hs, 042-deferred-20190918  |  Actual Points:
Parent ID:  #30200 | Points:  0.5
 Reviewer:  asn|Sponsor:
   |  Sponsor27-must
---+---
Changes (by dgoulet):

 * points:   => 0.5
 * milestone:  Tor: unspecified => Tor: 0.4.3.x-final


--
Ticket URL: 
Tor Bug Tracker & Wiki 
The Tor Project: anonymity online
___
tor-bugs mailing list
tor-bugs@lists.torproject.org
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs

Re: [tor-bugs] #31632 [Core Tor/Tor]: hs-v3: Service doesn't re-upload descriptor on circuit failure

[Tor Bug Tracker & Wiki]

#31632: hs-v3: Service doesn't re-upload descriptor on circuit failure
---+---
 Reporter:  dgoulet|  Owner:  (none)
 Type:  defect | Status:  new
 Priority:  Medium |  Milestone:  Tor:
   |  unspecified
Component:  Core Tor/Tor   |Version:
 Severity:  Normal | Resolution:
 Keywords:  tor-hs, 042-deferred-20190918  |  Actual Points:
Parent ID:  #30200 | Points:
 Reviewer:  asn|Sponsor:
   |  Sponsor27-must
---+---

Comment (by arma):

 For the v2 case, check out these elements in the descriptor struct:
 {{{
   /** Has descriptor been uploaded to all hidden service directories? */
   int all_uploads_performed;
   /** List of hidden service directories to which an upload request for
* this descriptor could be sent. Smartlist exists only when at least
 one
* of the previous upload requests failed (otherwise it's not important
* to know which uploads succeeded and which not). */
   smartlist_t *successful_uploads;
 }}}

 So v2 has the ability to notice that it e.g. doesn't have enough directory
 information about a given hsdir, and it will try republishing to just
 those hsdirs later in that case. But v2 is still missing the feature where
 the hsdir info is all set, and we make the circuit and attempt the upload,
 and something goes wrong with the circuit or stream.

 I looked at hs_circ_cleanup(), and maybe it is useful here but it's not
 obvious how.

 My thought instead would be to put something like the above v2 state into
 the v3 descriptor struct, and then hook
 connection_dir_client_request_failed() to note that a given hsdir upload
 needs to be retried on the next iteration. See how
 connection_dir_about_to_close(), which calls that request_failed function,
 has a call to connection_dir_client_refetch_hsdesc_if_needed(). We could
 expand that refetch function to be about refetching-or-reposting, or we
 could make a second parallel function that is about reposting if needed.)

--
Ticket URL: 
Tor Bug Tracker & Wiki 
The Tor Project: anonymity online
___
tor-bugs mailing list
tor-bugs@lists.torproject.org
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs

Re: [tor-bugs] #31632 [Core Tor/Tor]: hs-v3: Service doesn't re-upload descriptor on circuit failure

[Tor Bug Tracker & Wiki]

#31632: hs-v3: Service doesn't re-upload descriptor on circuit failure
---+---
 Reporter:  dgoulet|  Owner:  (none)
 Type:  defect | Status:  new
 Priority:  Medium |  Milestone:  Tor:
   |  unspecified
Component:  Core Tor/Tor   |Version:
 Severity:  Normal | Resolution:
 Keywords:  tor-hs, 042-deferred-20190918  |  Actual Points:
Parent ID:  #30200 | Points:
 Reviewer:  asn|Sponsor:
   |  Sponsor27-must
---+---
Changes (by neel):

 * status:  assigned => new


--
Ticket URL: 
Tor Bug Tracker & Wiki 
The Tor Project: anonymity online
___
tor-bugs mailing list
tor-bugs@lists.torproject.org
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs

Re: [tor-bugs] #31632 [Core Tor/Tor]: hs-v3: Service doesn't re-upload descriptor on circuit failure

[Tor Bug Tracker & Wiki]

#31632: hs-v3: Service doesn't re-upload descriptor on circuit failure
---+---
 Reporter:  dgoulet|  Owner:  (none)
 Type:  defect | Status:  assigned
 Priority:  Medium |  Milestone:  Tor:
   |  unspecified
Component:  Core Tor/Tor   |Version:
 Severity:  Normal | Resolution:
 Keywords:  tor-hs, 042-deferred-20190918  |  Actual Points:
Parent ID:  #30200 | Points:
 Reviewer:  asn|Sponsor:
   |  Sponsor27-must
---+---
Changes (by neel):

 * owner:  neel => (none)


--
Ticket URL: 
Tor Bug Tracker & Wiki 
The Tor Project: anonymity online
___
tor-bugs mailing list
tor-bugs@lists.torproject.org
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs

Re: [tor-bugs] #31632 [Core Tor/Tor]: hs-v3: Service doesn't re-upload descriptor on circuit failure

[Tor Bug Tracker & Wiki]

#31632: hs-v3: Service doesn't re-upload descriptor on circuit failure
---+---
 Reporter:  dgoulet|  Owner:  neel
 Type:  defect | Status:  assigned
 Priority:  Medium |  Milestone:  Tor:
   |  unspecified
Component:  Core Tor/Tor   |Version:
 Severity:  Normal | Resolution:
 Keywords:  tor-hs, 042-deferred-20190918  |  Actual Points:
Parent ID:  #30200 | Points:
 Reviewer:  asn|Sponsor:
   |  Sponsor27-must
---+---
Changes (by neel):

 * owner:  (none) => neel
 * status:  new => assigned


--
Ticket URL: 
Tor Bug Tracker & Wiki 
The Tor Project: anonymity online
___
tor-bugs mailing list
tor-bugs@lists.torproject.org
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs