Improve ergonomics on identity::Keypair
              
              #3623
            
            
              
  
  Closed
              
          
                  
                    
                      drHuangMHT
                    
                  
                
                  started this conversation in
                Ideas
              
            Replies: 2 comments 16 replies
-
| 
         We have  Two questions: 
  | 
  
Beta Was this translation helpful? Give feedback.
                  
                    16 replies
                  
                
            -
| 
         Also a pending draft PR that's probably worth noticing #2671.  | 
  
Beta Was this translation helpful? Give feedback.
                  
                    0 replies
                  
                
            
  
    Sign up for free
    to join this conversation on GitHub.
    Already have an account?
    Sign in to comment
  
        
    
Uh oh!
There was an error while loading. Please reload this page.
-
In #3350 we've made the decision to make
Keypairopaque. But I think there's more to be done. Currently deprecating keypair variants makes it harder to know what's inside. You have to try those methods one by one to see what it is. This won't be a problem if you're not importing and exporting keys. Instead of trying out many different methods to get the key imported or exported(which is annoying), we can unify them into two with one responsible for importing(bytes toKeypair) and one for exporting(Keypairto bytes). Matching can be done internally instead of letting users handle this. Plus, it's not a piece of performance critical code.Beta Was this translation helpful? Give feedback.
All reactions