Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove TC len_ingest vs max_frame_size check #431

Closed
Donnie-Ice opened this issue Mar 19, 2025 · 0 comments · Fixed by #432
Closed

Remove TC len_ingest vs max_frame_size check #431

Donnie-Ice opened this issue Mar 19, 2025 · 0 comments · Fixed by #432
Assignees
@Donnie-Ice
Labels
bug Something isn't working

Comments

@Donnie-Ice
Copy link
Contributor

Description

Wrong according to CCSDS TC Spec

Branch Name

No response

Reproduction steps

...

Screenshots



Logs




OS


Linux

      		
    

      
  





        



            

              
            

        

      


      
    








      

    



      

  
            

    

      
    

    


      

          @Donnie-Ice
Donnie-Ice




          added
  the 

  bug

  Something isn't working
 label


      Mar 19, 2025

    

  



    

    

      
    

    


      

          @Donnie-Ice
Donnie-Ice




              self-assigned this


      Mar 19, 2025

    

  



  


  
    
  
  

    
  Donnie-Ice 

      added a commit
      that referenced
      this issue

    
      Mar 20, 2025
    
    

      


  

    

      


  

      
        @Donnie-Ice
  





      

        
          [#431] Removed max frame size check, restored UT frame sizes to 1024
        


      


      

          

    
    
    

  

      


      

          

    
    
    

  

  

    
  



      


      
      

        
          bef4644
        
      

    

  




    

  




  

  

    
  


  

    


      @Donnie-Ice
Donnie-Ice



        linked a pull request
        Mar 20, 2025
        that will
        close
        this issue





  

    
      [nasa/cryptolib#431] Removed max frame size check, restored UT frame …
      #432
  



  

      
   Merged

  





  
  

      
        
      
      

  
    
    
  


    4 tasks
  





  




    

    


  


  
    
  
  

    
  Donnie-Ice 

      added a commit
      that referenced
      this issue

    
      Mar 25, 2025
    
    

      


  

    

      


  

      
        @Donnie-Ice
  





      

        
          Merge pull request #432 from nasa/431-remove-tc-len_ingest-vs-max_fra…
        

          
            
          

      


      

          

    
    
    

  

    


      


      

          

    
    
    

  

  

    
  



      


      
      

        
          01b7402
        
      

    

  

    

      
…me_size-check

[#431] Removed max frame size check, restored UT frame …

    




    

  













  
  

    

      

  





  




  
  

              

  
    Sign up for free
    to join this conversation on GitHub.
    Already have an account?
    Sign in to comment


  



  






  
                  




    

  


      
  

    Assignees
  



        

      

        

  
      @Donnie-Ice
  
    Donnie-Ice
    

      








      


  


  

    Labels
  



    

      

    bug

  Something isn't working








      

  

    

        

    Projects
  


        




    None yet




  



      


  

    
  

    Milestone
  


      No milestone





    
      
          


  

    

      
        

          
    

    Development
  




            
  
Successfully merging a pull request may close this issue.





    


  
          
      [nasa/cryptolib#431] Removed max frame size check, restored UT frame …

    
      nasa/CryptoLib
    





      
    

  




      

    

  
      

  

    

      1 participant
    

    

        
          @Donnie-Ice