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

Call to "stx_check_config_files.pro" can cause the startup script to hang #205

Closed
ennosigaeus opened this issue Jan 29, 2024 · 2 comments · Fixed by #206
Closed

Call to "stx_check_config_files.pro" can cause the startup script to hang #205

ennosigaeus opened this issue Jan 29, 2024 · 2 comments · Fixed by #206
Assignees
@ennosigaeus
Labels
bug Something isn't working

Comments

@ennosigaeus
Copy link
Member

Short Description
Säm Freeland noticed that the startup script hangs at "stx_check_config_files".

Details
Message from Säm:

Executive Summary: Please check SSL Certificate expiration date for {http,https}://dataarchive.stix.i4ds.net (?)
...
My test w/hang yesterday was via LMSAL server ; that means via LMCO proxy;   so at least one extra moving part compared to non-proxy SSW clients.
I repeated the stix startup test via a stanford sswidl client (solar3.stanford.edu) so "simpler" / No Proxy involved.
Still flagged an issue at the same line (160) of stx_check_config_files.pro - but instead of hanging did this:
Licensed for use by: Stanford University
Executing SSW IDL_STARTUP for: GEN
Executing SSW IDL_STARTUP for: STIX
% Compiled module: STX_CHECK_CONFIG_FILES.    <<< stix IDL_STARTUP
% Compiled module: DEFAULT.
... etc ...
% Compiled module: TEST_OPEN.
% IDLNETURL::GET:  CCurlException:  Error: Http Get Request Failed. Error =
              SSL certificate problem: certificate has expired, Curl Error   <<<<<
              Code = 60..
% Execution halted at: STX_CHECK_CONFIG_FILES  160
   /homeg/ssw/so/stix/idl/processing/spectrogram/stx_check_config_files.pro
%                      $MAIN$          
Executing SSW IDL_STARTUP for: SITE
... etc...
 
IDL>   ; << in spite of "Execution Halted" message, continues since all of the IDL_STARTUPs are run via IDL 'execute' <main_execute.pro>
 
The lmsal 'hang' v. stanford 'report/continue' may have same root cause (SSL certs?) , but LMCO-Proxy has less of a sense of humor.

Screenshots
N/A, see console log above.

Files

STIX-GSW/stix/idl/processing/spectrogram/stx_check_config_files.pro

Line 4 in c8810fd

; the energy lookup table (ELUT) files

Example Code
Simply start the environment using your standard SSW IDL script. NB: So far, only Säm was able to reproduce the issue.

Environment

  • IDL Version: Unknown
  • STIX GSW Version: v0.0.5
  • OS: Unknown
@ennosigaeus ennosigaeus added the bug Something isn't working label Jan 29, 2024
@ennosigaeus ennosigaeus self-assigned this Jan 29, 2024
ennosigaeus added a commit that referenced this issue Jan 29, 2024
@ennosigaeus
fixed #205 (fix by Dominic Zarro)
eda04fe
@samaloney
Copy link
Collaborator

I did notice before that the server only has SSL cert for https://pub099.cs.technik.fhnw.ch and not https://dataarchive.stix.i4ds.net

@ennosigaeus
Copy link
Member Author

I did notice before that the server only has SSL cert for https://pub099.cs.technik.fhnw.ch and not https://dataarchive.stix.i4ds.net

Hm, I wonder who set up the SSL certificate, @nicHoch or @drhlxiao.

@samaloney samaloney mentioned this issue Feb 1, 2024
Merged
grazwegian added a commit to grazwegian/STIX-GSW that referenced this issue Feb 2, 2024
@grazwegian
Merge branch 'i4ds'
adf4e23 
* i4ds:
  fixed i4Ds#205 (fix by Dominic Zarro)
  Update VERSION.txt
  Update stx_imaging_demo (i4Ds#203)
  Remove unused keyword from structure
  Add IDL_STARTUP file to stix/setup which will be run every time SSWIDL is started. Currently this only calls stx_check_config_files (i4Ds#196)
  Skip ELUT (i4Ds#182)
grazwegian added a commit to grazwegian/STIX-GSW that referenced this issue Feb 5, 2024
@grazwegian
Merge branch 'main' into default
6d2cfc3 
* main:
  fixed i4Ds#205 (fix by Dominic Zarro)
  Update VERSION.txt
  Update stx_imaging_demo (i4Ds#203)
  Remove unused keyword from structure
  Add IDL_STARTUP file to stix/setup which will be run every time SSWIDL is started. Currently this only calls stx_check_config_files (i4Ds#196)
  Skip ELUT (i4Ds#182)
grazwegian added a commit to grazwegian/STIX-GSW that referenced this issue Feb 5, 2024
@grazwegian
Merge branch 'main' into print-elut-file
d24a2df 
* main:
  fixed i4Ds#205 (fix by Dominic Zarro)
  Update VERSION.txt
  update elut index
grazwegian added a commit to grazwegian/STIX-GSW that referenced this issue Feb 5, 2024
@grazwegian
Merge branch 'i4ds' into skip-elut
5110eda 
* i4ds:
  fixed i4Ds#205 (fix by Dominic Zarro)
  Update VERSION.txt
  Update stx_imaging_demo (i4Ds#203)
  Remove unused keyword from structure
  Add IDL_STARTUP file to stix/setup which will be run every time SSWIDL is started. Currently this only calls stx_check_config_files (i4Ds#196)
  Skip ELUT (i4Ds#182)
grazwegian added a commit to grazwegian/STIX-GSW that referenced this issue Mar 21, 2024
@grazwegian
Merge branch 'main' into XSPEC-conversion
04f6b94 
* main: (45 commits)
  Add silent keyword and print ELUT name (i4Ds#201)
  Update of "stx_estimate_flare_location" (i4Ds#209)
  Automatic formatting
  Update VERSION.txt
  Lightcurve demo fix (i4Ds#208)
  Spectroscopy demo fix (i4Ds#207)
  fixed i4Ds#205 (fix by Dominic Zarro)
  Update VERSION.txt
  Update stx_imaging_demo (i4Ds#203)
  Remove unused keyword from structure
  Add IDL_STARTUP file to stix/setup which will be run every time SSWIDL is started. Currently this only calls stx_check_config_files (i4Ds#196)
  Skip ELUT (i4Ds#182)
  Bugfix release
  stx_read_fits : uncommented call to mrdfits to prevent compilation issue when getting version number
  preparing a new release
  Fixed bug in selection of energy indices of pixel data matrix (i4Ds#185)
  stx_subc_transmission: Improved accuracy of message displayed if simple_transm keyword is set.
  Update by Massa, P. - use simplified version of the subcollimator transmission for imaging (temporary solution)
  Some Improvement in aspect processing (i4Ds#179)
  Initial release of basic routines for downloading L2 AUX ephemeris files given a date and L1 science data fits files given a UID. (i4Ds#189)
  ...

# Conflicts:
#	stix/idl/processing/spectrogram/stx_convert_pixel_data.pro
#	stix/idl/processing/spectrogram/stx_convert_spectrogram.pro
#	stix/idl/processing/spectrogram/stx_fsw_sd_spectrogram2ospex.pro
#	stix/idl/processing/spectrogram/stx_write_ospex_fits.pro
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ennosigaeus ennosigaeus

Labels
bug Something isn't working
Projects
Planning Board for STIX GSW, Archiving and Processing
Status: Done
Milestone
No milestone
2 participants
@ennosigaeus @samaloney