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.

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

Fix EATON download #837

Closed
wants to merge 2 commits into from
Closed

Conversation

francois2metz
Copy link

@francois2metz francois2metz commented Jan 13, 2023

fix #836 and #829

I had to use a custom user agent, as akamai seems to forbid curl.

@SuperQ
Copy link
Member

SuperQ commented Jan 14, 2023

Ugh, really? They're blocking curl UA?

@SuperQ
Copy link
Member

SuperQ commented Jan 14, 2023

This needs a DCO sign-off. You can use git commit -s --amend to add it.

Signed-off-by: François de Metz <[email protected]>
Signed-off-by: François de Metz <[email protected]>
@h-leth
Copy link

h-leth commented Jan 27, 2023

Any progress on this PR?

@francois2metz
Copy link
Author

For some reasons the build fail on circleci. I tried to remove the manual user agent, but it fail later.

@h-leth
Copy link

h-leth commented Jan 28, 2023

Ok
I found the missing files in a Git repo and downloaded them from there. It works on my computer
If you want I can open a new PR or you can use my code and make a new commit

EATON_EPDU_URL    := https://raw.githubusercontent.com/librenms/librenms/master/mibs/eaton/EATON-EPDU-MIB
EATON_SENSOR_URL  := https://raw.githubusercontent.com/librenms/librenms/master/mibs/eaton/EATON-SENSOR-MIB

--snip--

$(MIBDIR)/EATON-EPDU-MIB.txt:
	@echo ">> Downloading EATON-EPDU-MIB.txt to $@"
	@curl $(CURL_OPTS) -o $@ $(EATON_EPDU_URL)

$(MIBDIR)/EATON-SENSOR-MIB.txt:
         @echo ">> Downloading EATON-SENSOR-MIB.txt to $@"
         @curl $(CURL_OPTS) -o $@ $(EATON_SENSOR_URL)

@h-leth h-leth mentioned this pull request Jan 30, 2023
@SuperQ
Copy link
Member

SuperQ commented Feb 14, 2023

Fixed with #843

@SuperQ SuperQ closed this Feb 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

EATON mib location is no longer valid
3 participants