mirror of
				https://github.com/juanfont/headscale.git
				synced 2025-11-04 10:01:05 +01:00 
			
		
		
		
	Merge pull request #237 from cure/preauthkeys-fix-default-expiration
Fix default preauthkey expiration
This commit is contained in:
		
						commit
						d944aa6e79
					
				@ -13,7 +13,7 @@ import (
 | 
				
			|||||||
)
 | 
					)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
const (
 | 
					const (
 | 
				
			||||||
	DefaultPreAuthKeyExpiry = 24 * time.Hour
 | 
						DefaultPreAuthKeyExpiry = 1 * time.Hour
 | 
				
			||||||
)
 | 
					)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
func init() {
 | 
					func init() {
 | 
				
			||||||
@ -145,14 +145,12 @@ var createPreAuthKeyCmd = &cobra.Command{
 | 
				
			|||||||
			Ephemeral: ephemeral,
 | 
								Ephemeral: ephemeral,
 | 
				
			||||||
		}
 | 
							}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
		if cmd.Flags().Changed("expiration") {
 | 
							duration, _ := cmd.Flags().GetDuration("expiration")
 | 
				
			||||||
			duration, _ := cmd.Flags().GetDuration("expiration")
 | 
							expiration := time.Now().UTC().Add(duration)
 | 
				
			||||||
			expiration := time.Now().UTC().Add(duration)
 | 
					 | 
				
			||||||
 | 
					
 | 
				
			||||||
			log.Trace().Dur("expiration", duration).Msg("expiration has been set")
 | 
							log.Trace().Dur("expiration", duration).Msg("expiration has been set")
 | 
				
			||||||
 | 
					
 | 
				
			||||||
			request.Expiration = timestamppb.New(expiration)
 | 
							request.Expiration = timestamppb.New(expiration)
 | 
				
			||||||
		}
 | 
					 | 
				
			||||||
 | 
					
 | 
				
			||||||
		ctx, client, conn, cancel := getHeadscaleCLIClient()
 | 
							ctx, client, conn, cancel := getHeadscaleCLIClient()
 | 
				
			||||||
		defer cancel()
 | 
							defer cancel()
 | 
				
			||||||
 | 
				
			|||||||
@ -426,7 +426,12 @@ func (s *IntegrationCLITestSuite) TestPreAuthKeyCommandWithoutExpiry() {
 | 
				
			|||||||
	assert.Nil(s.T(), err)
 | 
						assert.Nil(s.T(), err)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	assert.Len(s.T(), listedPreAuthKeys, 1)
 | 
						assert.Len(s.T(), listedPreAuthKeys, 1)
 | 
				
			||||||
	assert.True(s.T(), time.Time{}.Equal(listedPreAuthKeys[0].Expiration.AsTime()))
 | 
					
 | 
				
			||||||
 | 
						assert.True(s.T(), listedPreAuthKeys[0].Expiration.AsTime().After(time.Now()))
 | 
				
			||||||
 | 
						assert.True(
 | 
				
			||||||
 | 
							s.T(),
 | 
				
			||||||
 | 
							listedPreAuthKeys[0].Expiration.AsTime().Before(time.Now().Add(time.Minute*70)),
 | 
				
			||||||
 | 
						)
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
func (s *IntegrationCLITestSuite) TestPreAuthKeyCommandReusableEphemeral() {
 | 
					func (s *IntegrationCLITestSuite) TestPreAuthKeyCommandReusableEphemeral() {
 | 
				
			||||||
 | 
				
			|||||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user