quickstarts icon indicating copy to clipboard operation
quickstarts copied to clipboard

.NET SDK: Cryptography Quickstart example

Open msfussell opened this issue 1 year ago • 9 comments

Once this issue gets checked in https://github.com/dapr/dotnet-sdk/pull/1217 then there also needs to be an example for the Quickstart to show how to use this.

This issue also has to align with the docs issue https://github.com/dapr/docs/pull/3947

msfussell avatar Jan 22 '24 06:01 msfussell

/assign

WhitWaldo avatar Jan 22 '24 12:01 WhitWaldo

@msfussell For purposes of setting up the SDK quickstart, can I assume that the crypto bits will be included in the 1.13 release?

WhitWaldo avatar Jan 22 '24 12:01 WhitWaldo

@WhitWaldo - Assigned to you and yes to the SDK inclusion in the v1.13 release

msfussell avatar Jan 23 '24 05:01 msfussell

Experiencing an issue with the VS debugger while testing this, but working on it. Will have some code to review once I can conform it's working all the way through.

WhitWaldo avatar Jan 24 '24 19:01 WhitWaldo

@msfussell and @philliphoff do you have a moment to help WhitWaldo with the vs debugging issue so we can wrap this up?

paulyuk avatar Feb 21 '24 17:02 paulyuk

@paulyuk @WhitWaldo - Yes, tell us your issue. @philliphoff may be the best person if this is a VS Debugger issue.

msfussell avatar Feb 21 '24 17:02 msfussell

Happy to help, though I'm out of the office this week.

philliphoff avatar Feb 21 '24 22:02 philliphoff

Today it's tax day, but I'll see if I can't wrap this up later this evening/tomorrow.

WhitWaldo avatar Feb 21 '24 23:02 WhitWaldo

@paulyuk @msfussell @philliphoff Ok, so I've been poking at this for a few minutes trying to figure out where the stumbling block is. The problem spans several pieces:

  1. Can't build the crypto quickstart program itself because it relies on a NuGet package that doesn't yet exist (which makes it an odd requirement for this release). So I'm accommodating by instead building the crypto example since that has a direct reference to the Dapr package.
  2. I've already written an example on the Cryptography branch that demos this using a string and a file stream example. I was just going to tweak this to build the quickstart. This worked at some point, but there's no easy way I've found to build an E2E test that does what these examples do, so it broke and I didn't realize it did. My little spot-tests work fine, but they're hardly testing the whole thing.
  3. Something is wrong with the example as it's timing out with task cancelled exception. This is my current blocker is figuring out whether it's a problem with the example (not likely - it doesn't do much) or with the Dapr crypto changes (likely, this is the only thing that I've changed). But it's a work in progress to figure out what's wrong. And that's the latest.

WhitWaldo avatar Feb 22 '24 22:02 WhitWaldo