No Data since today's Alpha ESS Update

Hi,
Since the update from Alpha ESS, the data query no longer works.
I get the following error:
2023-11-04 09:46:13.375 ERROR (MainThread) [alphaess.alphaess] 'data'
2023-11-04 09:46:13.376 ERROR (MainThread) [custom_components.alphaess] Unexpected error fetching alphaess data: 'data'
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 290, in _async_refresh
self.data = await self._async_update_data()

File "/config/custom_components/alphaess/coordinator.py", line 30, in _async_update_data
jsondata: json = await self.api.getdata()

File "/usr/local/lib/python3.11/site-packages/alphaess/alphaess.py", line 154, in getdata
units = await self.__get_data("Account/GetCustomMenuESSList")

File "/usr/local/lib/python3.11/site-packages/alphaess/alphaess.py", line 176, in __get_data
if not await self.__connection_check():

File "/usr/local/lib/python3.11/site-packages/alphaess/alphaess.py", line 147, in __connection_check
return await self.authenticate(self.username, self.password)

File "/usr/local/lib/python3.11/site-packages/alphaess/alphaess.py", line 67, in authenticate
if "AccessToken" in json_response["data"]:

KeyError: 'data'
2023-11-04 09:46:13.379 DEBUG (MainThread) [custom_components.alphaess] Finished fetching alphaess data in 0.525 seconds (success: False)

Thank you for looking into that issue!

It appears that Alpha ESS has updated its API authentication protocol, transitioning from a simple password-based login to a more secure method that involves hashing the password prior to login. Additionally, the login endpoint has been modified; what was previously accessed via https://cloud.alphaess.com/api/Account/Login has now been changed to https://cloud.alphaess.com/api/base/user/login.

Code response able for creating the encrypted password.
Hope this helps.
export function encrypt(password, username) {
let aeskey = CryptoJS.SHA256(username)
let iv = CryptoJS.MD5(username)
let encrypted = CryptoJS.AES.encrypt(password, aeskey, {
iv: iv,
mode: CryptoJS.mode.CBC,
padding: CryptoJS.pad.Pkcs7,
})
return encrypted.toString()
}

password is sent encrypted to https://cloud.alphaess.com/api/base/user/login (see @MorneSaunders360)
Header "Authorization" is not "Bearer eyJ..." any more, its without Bearer prefix now
Header "Authsignature" and "Authtimestamp" is gone

It seems some of you know how to get the integration to work again.
Can anyone advise what I would need to do in my home Assistent or in the integration files to fix the issue, greatly appreciated.

Someone needs to make a pull request with these changes, and hopefully it gets pulled, packaged and updated by the repo owner

If it's doable then Charles is usually pretty quick to fix, just hang tight.

I’ve posted a quick fix on an issue to the alphaess library, hopefully Charles can publish a new version which will sort it temporarily.
Alpha are trying to push people to use their new public api but from what I’ve seen is still unstable and doesn’t have all functionality

Thanks Matt!  I can get this pulled and an update out later today.  Much appreciated.

Charles and other contributors all do this voluntarily and are usually very good and diagnosing issues, updating libraries, testing and issuing an update to the HA integration.
Everyone needs to be respectful of their time and efforts. If people don't understand that this is the way that open source software works then HA is probably the wrong home automation solution for them.
Charles, Matt et al - thank you so much for your continued work on this integration and for putting up with the challenges of working with AlphaESS.

I've pushed a beta update to through HACS that uses @matt-oneill 's temporary workaround of changing the base URL for the API.  I'm going to work on incorporating the changes identified by @MorneSaunders360 and @ckarrie

Nice workaround! Thank you!

Nice one Charles. I had a look at the other apis earlier and a lot of them have changed the path so not a simple blanket change to the login endpoint.
Maybe it’s time to re visit the new api

Thanks folks, it’s working for me.

Nice one Charles. I had a look at the other apis earlier and a lot of them have changed the path so not a simple blanket change to the login endpoint. Maybe it’s time to re visit the new api

I've found several equivalents, most of them in fact but the missing one for me is the Discharge numbers - I used to grab them from system statistics but I cannot see any mention of discharge in any of the results I'm getting back in the web interface now.....

I'm sorry to be a pain but what do I need to do to get the fix applied? I'm not an expert at the internals of HA, just an enthusiastic amateur.

I'm sorry to be a pain but what do I need to do to get the fix applied? I'm not an expert at the internals of HA, just an enthusiastic amateur.

Same for me, I have the same question.

If you deployed using HACS then should just be a matter of finding the AlphaESS repo in HACS and click on update information in the  three dots menu.  Navigate back to the main list of HACS integrations and you should see an update button for AlphaESS.

If you deployed using HACS then should just be a matter of finding the AlphaESS repo in HACS and click on update information in the three dots menu. Navigate back to the main list of HACS integrations and you should see an update button for AlphaESS.

Thanks, you put me on the right track, sort of. I went into HACS, clicked on the Alpha Integration then from the  Menu selected Redownload with the "show Betas" option, then selected the latest Beta you released.
An HA restart and all seems good.
Thanks for your help and the integration. much appreciated.
@weissjuergen , hope this helps

@weissjuergen , hope this helps

Yes, it did, I can see my entities again. Thank you Charles !

Amazing! Thank you so much! Ist there any oportunity to support your work with money? Would be a pleasure for me.

Amazing! Thank you so much! Ist there any oportunity to support your work with money? Would be a pleasure for me.

No need for any cash - I do this because I have my own solar system to manage.  This fix is only temporary, so don't be alarmed if it doesn't last for long - it's dependent on Alpha being slow to update another URL but I think we all know that's unlikely to last forever.

Thank you Charles.  However, updating to the latest beta version did not work for me here in Australia.  Will wait until you release a fix.

I downloaded the beta version (v0.3.4beta) and it is working correctly again. Thanks Charles!

Just updated to the BETA and it's all working again. Thanks for fixing this so quickly.

+1, BETA looks good. To all the experts who know what they're doing with this, big thumbs up, and to you Charles for the time and effort to bring this to us all. HA is great but nothing without people to bring us the goodies!

All was working fine since the temp fix. A few min ago the Connection ist broken again. Can you check your systems and see this issue is wide spread or just me.

Appears to be working, check your systems connection to the cloud, seems to be a issue on your end.

Unfortunately I still can't get it to Configure the integration,  It gives Unknown Error after entering UserName and Password and the logs show the following:
Logger: custom_components.alphaess.config_flow
Source: custom_components/alphaess/config_flow.py:66
Integration: Alpha ESS (documentation)
First occurred: 14:21:23 (3 occurrences)
Last logged: 14:24:50
Unexpected exception
Traceback (most recent call last):
File "/config/custom_components/alphaess/config_flow.py", line 66, in async_step_user
info = await validate_input(self.hass, user_input)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/alphaess/config_flow.py", line 44, in validate_input
if not await client.authenticate(data["username"],data["password"]):
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/alphaess/alphaess.py", line 60, in authenticate
if "AccessToken" in json_response["data"]:
~~~~~~~~~~~~~^^^^^^^^
KeyError: 'data'
I expect I still have remants of the old version on my system, but I can not seem to find out how to clean it out and start again.  I have removed the integration previously and added the repository back in today, but setting up the integration gives the same error.   I must be doing something wrong, but have not idea what.   Any hints on how to get the new version sorted in this scenario?

scenario?

It's maybe worth trying to completely remove the component and the associated library manually and then trying again.
I think the library can be deleted by removing the entire folder /usr/local/lib/python3.11/site-packages/alphaess
The component can be deleted by removing the sub-folder custom_components/alphaess/ under your home assistant configuration folder.

Our next version of Web API will add anti-crawling function. Please move to OpenAPI, otherwise you will update with us every time.Thanks。Open api we are slowly optimizing the request limit。

Challenge accepted

I have a working version of the component using the AlphaESS OpenAPI running in test.  Before I consider pushing it out as an updated release I want to give everyone a heads up that the Open API will require you to register with the Alpha ESS developer portal and obtain an AppID and an AppSecret for yourself and to also bind your own inverter to that AppID in the portal.
If anyone hasn't already registered with the AlphaESS OpenAPI portal then this component will stop working when I push the updated release.
I will put a notification on the main page of the GitHub repository as well but for everyone following this thread - please register with https://open.alphaess.com/ as soon as you are able.

done, looking forward to a more reliable and stable connection.
It looks like they (AlphaESS) are having other software problems as well in their monitoring. Currently my wallbox usage isn’t reported properly anymore - and haha, they told me to check the correct cabling.

Am 07.11.2023 um 13:37 schrieb CharlesGillanders @.***>:
I have a working version of the component using the AlphaESS OpenAPI running in test. Before I consider pushing it out as an updated release I want to give everyone a heads up that the Open API will require you to register with the Alpha ESS developer portal and obtain an AppID and an AppSecret for yourself and to also bind your own inverter to that AppID in the portal.
If anyone hasn't already registered with the AlphaESS OpenAPI portal then this component will stop working when I push the updated release.
I will put a notification on the main page of the GitHub repository as well but for everyone following this thread - please register with https://open.alphaess.com/ https://open.alphaess.com/ as soon as you are able.
—
Reply to this email directly, view it on GitHub https://github.com/CharlesGillanders/homeassistant-alphaESS/issues/76#issuecomment-1798418356, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABCHOA6NMO2LZXFQYQGC5FDYDITQFAVCNFSM6AAAAAA65N3WICVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOJYGQYTQMZVGY.
You are receiving this because you are subscribed to this thread.

That really gets things moving. The registration is complete. Ready when you are!
And again: Thanks a lot!

 When I try and add mine it asks for the system S/N which I guess is the one beginning AL rather than the actual Inverter S/N? I have both. But then it wants  CheckCode which is where I get unstuck. Anyone any idea what this is pls? I assume after entering that I may get the Verification Code.

It’s on the side of the unit.

It’s on the side of the unit.

Well I'll be!
Never thought to look at the hardware itself. Doh!
Thanks for that

Registered. Thanks for the ongoimng work

Looking very much forward to the fix. Realized (for the last ~week it has been down) that this is the most important integration in my HA setup :D
Didn't try the "beta", but will try out the fix as soon as HACS gets the update :)

I have published release 0.4.0 of the HomeAssistant custom component - this is the switch to using the official OpenAPI.
There are some details on how to switch an existing deployment to use the new API in the Github README https://github.com/CharlesGillanders/homeassistant-alphaESS/blob/main/README.md

Downloaded and it works for me.
Huge thanks Charles

Works like a charm.
One minor issue: The dialog to add the credentials does not have any kind of description for me. All I saw were 2 unnamed input fields.

Working like a dream, thank you for all the efforts!
On Tue, Nov 14, 2023 at 12:41 PM CharlesGillanders @.***>
wrote:

I have published release 0.4.0 of the HomeAssistant custom component -
this is the switch to using the official OpenAPI.
There are some details on how to switch an existing deployment to use the
new API in the Github README
https://github.com/CharlesGillanders/homeassistant-alphaESS/blob/main/README.md
—
Reply to this email directly, view it on GitHub
https://github.com/CharlesGillanders/homeassistant-alphaESS/issues/76#issuecomment-1809957508,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZ6DF4RGZIJ4CEITBKZN5J3YENDFZAVCNFSM6AAAAAA65N3WICVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMBZHE2TONJQHA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***
com>

--
Kind regards
Reece
Reece BotesCTO
Office 7 | Inospace | 5 Sunrise Circle | Ndabeni
O +27 21 424 8511 <+27%2021%20424%208511>  M +27 82 9303130
<+27%2082%20930%203130>
E @.*  W www.kinektek.com

Thanks Charles, all working.

Thanks @CharlesGillanders great work.
Same for me  - but its working again hopefully we won't have to keep chasing api changes

Thanks for this update
I think I've found a problem when calling the Alpha ESS: Set Battery Charge
service, It just returns an unknown error when I call it.
On Tue, 14 Nov 2023 at 10:41, CharlesGillanders @.***>
wrote:

I have published release 0.4.0 of the HomeAssistant custom component -
this is the switch to using the official OpenAPI.
There are some details on how to switch an existing deployment to use the
new API in the Github README
https://github.com/CharlesGillanders/homeassistant-alphaESS/blob/main/README.md
—
Reply to this email directly, view it on GitHub
https://github.com/CharlesGillanders/homeassistant-alphaESS/issues/76#issuecomment-1809957508,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5GM6ZCNIPQVMJCP2MNAGGTYENDFTAVCNFSM6AAAAAA65N3WICVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMBZHE2TONJQHA
.
You are receiving this because you commented.Message ID:
@.***>

I haven't yet tried to call the battery services (I use an automation later in the evening)

Thanks for this update I think I've found a problem when calling the Alpha ESS: Set Battery Charge service, It just returns an unknown error when I call it.
…
On Tue, 14 Nov 2023 at 10:41, CharlesGillanders @.> wrote: I have published release 0.4.0 of the HomeAssistant custom component - this is the switch to using the official OpenAPI. There are some details on how to switch an existing deployment to use the new API in the Github README https://github.com/CharlesGillanders/homeassistant-alphaESS/blob/main/README.md — Reply to this email directly, view it on GitHub <#76 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/A5GM6ZCNIPQVMJCP2MNAGGTYENDFTAVCNFSM6AAAAAA65N3WICVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMBZHE2TONJQHA . You are receiving this because you commented.Message ID: @.>

Would you mind opening a new issue to track that separately please?

Issue opened as requested.

im using the new API (outside of home assistant), the set charge is a PUT call now
https://github.com/DarkSlice1/AlphaEss_Monitor/blob/dumpZappi/src/main/scala/api/alpha/alpha.scala#L108

AWESOME to have this integration running again!
As matt-oneill mentioned I also didn't have any labels in the "credentials prompt".

But the most important thing is that it works.
I went from having 21 to 23 entities after adding the new "integration entry", don't know what the new ones are but everything works as before :)
Thanks for the great work Charles!

I can confirm that the new version of the integration works as expected.
Well done Charles!

Thank you Charles, all working on the new version.

But the most important thing is that it works. I went from having 21 to 23 entities after adding the new "integration entry", don't know what the new ones are but everything works as before :)

The two new entities are additional PPV3, and PPV4 real time PV power generation figures - I don't know which AlphaESS inverters support four strings of PV (my install only has two) but if you are lucky enough to have them the new API displays them.

I haven't yet tried to call the battery services (I use an automation later in the evening)

There was an error with changing the charge/discharge settings - please make sure to update to 0.4.1 which fixes this.


docs: document color utils

I followed your advice, everything is up to date.

Thanks for your contribution, I have moved this into a separate section to help keep things cleaner in the future.


FULL-FREE!! Will Levis And Gia Duddy Leaked Video Viral On Social Media X Twitter[5U5US]

⭐⭐⭐⭐⭐
12 sec ago Will Levis And Gia Duddy Leaked Video Viral On Social Media.FULL-FREE Will Levis And Gia Duddy Leaked Video Viral On Social Media X Twitter[9U9US]
..
🔴►►https://leakedvideo.co/free-video/
..
✅🔴👉 https://leakedvideo.co/free-video/
..
..

..
..

..
..

..
..
1 seconds ago — Where i can watch Will Levis And Gia Duddy Full Video? WATCH HERE! Viral Will Levis And Gia Duddy Full Video Instagram, Twitter (X), Discord. Will Levis And Gia Duddy video took the internet by storm and amazed viewers on various social media platforms. Will Levis And Gia Duddy, a young and talented digital creator, recently became famous thanks to this interesting video. Will Levis And Gia Duddy Spiderman video instagram
Where i can watch Will Levis And Gia Duddy Full Video? WATCH HERE! Viral Will Levis And Gia Duddy Full Video Instagram, Twitter (X), Discord. Will Levis And Gia Duddy video took the internet by storm and amazed viewers on various social media platforms. Will Levis And Gia Duddy, a young and talented digital creator, recently became famous thanks to this interesting video. Will Levis And Gia Duddy Spiderman video Instagram
Will Levis And Gia Duddy Leaked Video Viral On Social Media..Will Levis And Gia Duddy Video Viral On Social Media.Will Levis And Gia Duddy Video Viral On Social Media.Will Levis And Gia Duddy Video Viral On Social Media.
Will Levis And Gia Duddy Video Viral On Social Media 2024.Will Levis And Gia Duddy Video Viral On Social Media 2024.
The recurrent theme of leaked tapes and the subsequent fallout serves as a reminder of the fragility of reputation in the digital era.


Attachments - Can we do something smart so they can be viewed in App?

I.E dont need to be downloaded and can have text captions?
open to all ideas on this one

There are a lot of libs for previewing pdf files without downloading them https://projects.wojtekmaj.pl/react-pdf/

Seeing as how browsers have excellent support for viewing both pdf and txt files I think we can change the behavior to open the attachment in a new browser tab. This will probably feel way more smooth to the user

Looks like this when you open a pdf in native edge

This is a better solution than the current one IMO in any case.
Dev context:
The attachments are all tagged as octet-stream so we need to set the correct file headers

Lets do this in the next version


[Zeppelin-3307] - Improved shared browsing/editing for the note

What is this PR for?
Now if the note is opened in several tabs (or several users are watching or editing it), then there may be problems. Loss of code entered by the user, reset the cursor position.
This PR adds a basic opportunity for collaborative editing. For the organization of joint editing, the library diff-match-patch is used.
PR does not change the logic of operation if the note is used by one person.
Also, maybe this will solve the problem with ZEPPELIN-3131.
What type of PR is it?
Improvement
What is the Jira issue?
ZEPPELIN-3307
Screenshots (if appropriate)

Questions:

Does the licenses files need update? no
Is there breaking changes for older versions? no
Does this needs documentation? no

@prabhjyotsingh @r-kamath
could you help with the review?

Zeppelin has websockets and two users can view a note simultaneously, but there is a bug.

Open the note in two tabs.
Write anything in first tab for second.
Wait for 10 seconds.
Repeat the last two steps several times.
Check that written code is lost.

Thus paired coding is painful.
Please review this PR.

I think this PR might be not bad for a small number of users, but, I'm not sure users increase. What do you think of it? It looks like calling websockets everytime whenever all users make changes

@jongyoul, yes sending any change to the server is a big load. But this will happen only if the notebook is opened by several users. The current implementation is subject to errors in consequence of which users may lose some of the code. Even if the second user just open the tab in the browser. This is very annoying.

@prabhjyotsingh @r-kamath Could you help take a look at it ? It seems a necessary feature.

@Savalek I agree with unexpected ignorances feel bad. Can you add test cases both of backend and frontend? And please make sure the relationship with personalized mode.
This is an idea but needs to think of how we improve this feature? If this mode is disabled, all writers can write in the same way? otherwise, prohibit it except owner? When personalized mode is enabled, how does this feature behave?

@jongyoul, I have corrected.

@Savalek Thanks for this new feature. IMHO it will be nice to have an option to turn on/off the collaborative mode and a way to show the number of users collaborating.
something like  with a proper tooltip.

@r-kamath, what do you mean by turn off/on option?
This option will be global and enable/disable will only be available for the server administrator?

@jongyoul, I do not like it when try {...} catch {...} is used many times in the code. Therefore, I slightly changed the handling of ClassCastException. Could you look at the new implementation and say what you think about it?

@Savalek Thanks!! That’s what I want to do exactly. I also don’t like to use try and catch several times.

@jongyoul, I added tests.

@Savalek Thanks for adding e2e test but can you add also functional test for your backend code? It would be better to make a mock for NotebookServer to check if broadcast is called properly.

@r-kamath, @prabhjyotsingh, i fixed the description

@jongyoul, I added tests for backend.

@Savalek BTW, https://travis-ci.org/Savalek/zeppelin/builds/380644746 Can you check your Travis? Zeppelin has a lot of tricky tests but, in your case, it's a bit different. Do you know the reason?

@jongyoul, I run two tests. "master" and "zeppelin-3307 merge master." Travis shows the same result. Errors in this tests identical.

Will merge it if there's no further discussion

Fixed conflicts. What about merge?

This feature could replace "personalized mode". I will create PR to remove "personalized mode". Thank you @Savalek again.

@jongyoul As I understand personal mode allows users run paragraphs and have different views and different results due user chosen values in dynamic forms.
I'm against the removal of personal mode.

@mebelousov The main purpose of "personalized mode" is to keep the current user's view. But if the user refreshes the browser, it changes the newest one. Do you think it's enough?

@jongyoul I see next usecase. 10 users open the note, put client ID in dynamic form, refresh note, get and process data  result. In this case getting of default note view is OK.
That is in personal mode we may not save note updates.

Git it. Thanks. I thought it might not be useful. but I might be wrong. Thank you.


Allow users to reload and automatically reapply settings modified by set

problem
I agree this is not a major feature, but I find it important for sbt to be consistent.
Sbt provides lots of functionality around the session and the ability to control the modifications to the build that have been done via set. Let's see it:
clear, clear-all

	Removes temporary settings added using 'set' and re-evaluates all settings.
	For 'clear', only the settings defined for the current project are cleared.
	For 'clear-all', all settings in all projects are cleared.

list, list-all

	Prints a numbered list of session settings defined.
	The numbers may be used to remove individual settings or ranges of settings using 'remove'.
	For 'list', only the settings for the current project are printed.
	For 'list-all', all settings in all projets are printed.

remove <range-spec>

	<range-spec> is a comma-separated list of individual numbers or ranges of numbers.
	For example, 'remove 1,3,5-7'.
	The temporary settings at the given indices for the current project are removed and all settings are re-evaluated.
	Use the 'list' command to see a numbered list of settings for the current project.

save, save-all

	Makes the session settings permanent by writing them to a '.sbt' configuration file.
	For 'save', only the current project's settings are saved (the settings for other projects are left alone).
	For 'save-all', the session settings are saved for all projects.
	The session settings defined for a project are appended to the first '.sbt' configuration file in that project.
	If no '.sbt' configuration file exists, the settings are written to 'build.sbt' in the project's base directory.

In my personal case, I always have a long-running instance of sbt. I use it across different work days. During my work session, I do modifications to the build that I want to persist across the session, no matter if I reload. Reloading is required to add new plugins, modify versions of project dependencies, etc.
When I'm forced to reload, I usually spent some time finding the previous set commands and applying them one by one (note that doing so in just one set instruction is possible but not often since the set executions are incremental and unrelated to each other).
expectation
Right now, reload reloads the build from a clean slate and throws away my modified settings "at runtime" via set.
I'd like that sbt includes a way for me to persist these settings across different reloads. That can come in two different shapes (they are not necessarily exclusive, they can both be implemented if deemed appropriate):

Have a global setting that I can turn on and that will not throw these settings away.
Add a session reload that will do a reload keeping my current session.

sbt version: all sbt versions.

During my work session, I do modifications to the build that I want to persist across the session, no matter if I reload.

Could you describe your typical work day set scenario as repro steps? Do you enable compiler flag or do you bring in library dependencies?
This might be a tricky one if we are partly depending on this amnesia for things like ++ to work. Let's say you've ran:
> set scalaVersion := "2.11.11"
> ++2.12.3

which one wins?

Could you describe your typical work day set scenario as repro steps? Do you enable compiler flag or do you bring in library dependencies?

Those last two are common use cases. I also change scala versions a lot, and I want them to be persisted. I find that the feature I'm adding is especially useful when fiddling with interactions between the settings that i've just added to the build (like a compiler plugin) and the settings I've added via set.

This might be a tricky one if we are partly depending on this amnesia for things like ++ to work. Let's say you've ran.

Good point. In my view, ++2.12.3 is just an alias of set scalaVersion := "2.12.3". The implementation of the feature I'm proposing would treat it as such. I think this is the only command that is problematic.

I agree this would be a useful feature to add. I propose we add it as a flag to reload. Perhaps reload keep-session or some such.

/cc @edmundnoble who could've used this too when he wanted to add (and then remove) some session settings) and use reload at the same time, in a custom command - having a reload keep-session would've given him more options.


Change language on "mouse-pointer" blocks

Overview
Scratch 3.0 will work on a wide variety of devices including Chromebooks, Windows Surface, iPads, and Android tablets. Many of these devices either exclusively support touch or support a mixture of touch and mouse interaction. Because of this, we would like to retire use of the terms "mouse-pointer" and "clicked" and use the more generic terms "pointer" and "tapped". This will impact a large number of blocks, but feels important in order to properly support touch platforms moving forward.
Changes
Move

[x] go to (pointer)
[x] glide (1) secs to (pointer)
[x] point towards (pointer)

Events

[x] when this sprite tapped
[x] when stage tapped

Sensing

[x] <pointer down?>
[x] (pointer x)
[x] (pointer y)
[x] (distance to (pointer))

Details

[x] The (distance to (pointer)) block should be moved down to below the (pointer y) block. It's current positioning in the "Sensing" category doesn't make much sense (particularly for a block that isn't used very heavily).

Moved from LLK/scratch-vm#1015
/cc @thisandagain @speakkvisually @kenny2github

I wonder if we will see any confusion related to the word "touching" on touch devices?
touching (sprite)?
touching (color)?
(color) is touching (color)?
Not sure if it would make sense to rename these though. One idea would be to change just the first two like this (though these are longer, unfortunately):
this sprite is touching (sprite)?
this sprite is touching (color)?

@ericrosenbaum I share the concern. Should we file that as a follow-up item? It would be good to talk through these issues with @ntlrsk.

Between negative feedback from resource partners that came in today (major changes to blocks coming in too late) and some of the questions / concerns around this I have closed the PR and recommend we table this for discussion post release.
/cc @carljbowman @ntlrsk @ericrosenbaum


Extension stopped adding View Issue on GitHub tooltip on individual email list entries

Now, I have to open the email in Gmail t get a link to the issue. This is still way better than nothing, but it adds an extra step.
I am using Vivaldi 3.5 (based on chromium) on kubuntu 18.04.

The extension stopped working for me like a week ago I guess. Usually I used the buttons in the mail, but now they are greyed out. /cc @muan

@josephj11  That feature is dependent on the Gmail feature "go to action"

If that button isn't present then there's no way for the extension to read the URL. Looks like GitHub or Gmail no longer does this. At least judging from the notification emails that I have in my inbox currently.

@xh3b4sd Should be fixed shortly. (once the extension update goes through)

@muan Thank you for fixing this. Would you mind submitting a new version to addons.mozilla.org as well?

@wagnerand Done. Thanks for reminding me.

Works for me again. Thank you for your responsiveness. Appreciate the work a lot. ❤


Some implementations

Just some quick and easy implementations:

Added a conditional_check_verify function functions to Groth16Verifier and Gm17Verifier Gadgets.
Implemented ToBitsGadget for UInt8 and Vec.
Added a kary_or function to Boolean.

Thanks for the PR!


How is it possible use CUPS as client inside a docker container?

I have a application which uses lp command, by deploying it to a container I realized problems to print using CUPS.
Dockerfile:
FROM node:15

RUN apt-get update && \
    apt-get -y install cups cups-pdf

WORKDIR /app/

COPY package*.json ./

RUN npm install

COPY . .

EXPOSE 8080

CMD ["npm","run","proxy"]

Docker-compose:
version: '3'
services:
  ambiente_teste_project:
    build: .
    ports:
      - "8080:8080"
    container_name: teste_container
    volumes:
      - /var/run/cups/cups.sock:/var/run/cups/cups.sock

That attempt I got to run lp commands, this output is the same in my computer (Linux Ubuntu), both printers is working.
root@2911c9cd355e:/app# lpstat -p
printer CUPS-PDF-Printer is idle.  enabled since Wed Dec  9 21:40:59 2020
printer iR1730-1740-1750-UFRII-LT now printing iR1730-1740-1750-UFRII-LT-57.  enabled since Wed Dec  9 21:29:55 2020
        The printer is not responding.

Trying to print in both printer, I opened "Printers" at HostServer (Linux Ubuntu) and realized which none of printers is receiving any job.
lp -o landscape -o fit-to-page -o media=A4 src/assets/large.pdf -d CUPS-PDF-Printer
Anyone has suggestion to work Cups Client?

Please report this to the OpenPrinting CUPS project. We are working on getting containerized printing working better for v2.4.0.


Refactor ActionT to use ReaderT pattern

scotty-0.20 :

Rewrite ActionT using the "ReaderT pattern" : https://www.fpcomplete.com/blog/readert-design-pattern/
Clarify the exception handling mechanism of ActionT, ScottyT.  The StatusError type can be thrown and caught, but Next, Finish and Redirect cannot be caught by the user. For StatusError and any user-defined (or imported) exception types, one can either catch exceptions inline with rescue or globally for the whole app with defaultHandler.

Breaking changes:

Introduce unliftio as a dependency, and base exception handling on catch.
rescue changes signature to use proper Exception types rather than strings.
All ActionT methods (text, html etc.) have now a MonadIO constraint on the base monad rather than Monad because the Response is in a TVar inside ActionEnv. rescue has a MonadUnliftIO constraint.  The Alternative instance of ActionT is also based on MonadUnliftIO because <|> is implemented in terms of catch.

Closes #310 , closes #309 in passing
closes #110 , closes #153

it works omg it w o r k s I'm so happeeeehhh

Noice

@ocramz Is it intentional that you merged this PR? Before merging this PR, At very least I expected you to

Clean up fixme/WIP commits by git rebase
Wait for other maintainers' approval

@fumieval yes sorry for not waiting but I triple checked everything, added more tests around the new exception mechanism and made a Hackage release. These were long-due changes anyway..


⚠ Bofrost.de has degraded performance

In a19809b, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1389 ms

Resolved: Bofrost.de performance has improved in c9929d5 after 6 minutes.


feat(sast_warnings): add invalidSemVer translations

As part of PR 139 https://github.com/NodeSecure/scanner/pull/139 it was necessary to create an invalidSemVer token for the French and English languages

@fabnguess Can you check for the broken test ? It's not related to your changes but it should not be hard to be fixed

ok i'm looking at this

@fabnguess You can rebase #97 once it's merged ;) I wanted to udpate deps anyway so I fixed it

Hi @PierreDemailly , well noted, I will keep an eye disappointed


Loading additional .js files in Electron

The documentation seems to be missing information on this.
I've added jQuery to my project as described here. It seems that to load additional .js scripts into a window you have to use require to get them, as loading Twitter Bootstrap in a <script> tag causes Uncaught Error: Bootstrap's JavaScript requires jQuery error, even though jQuery is loaded. How exactly do I do this?
Thanks in advance!

You should put the <script> tag after require('jquery');.


Importent update make can-globals 1.0

If you rebase with master this should pass now i think.

@matthewp done


Update OpenEBS Architecture

Signed-off-by: ShyaniPal<EMAIL_ADDRESS>

Flow to Volume exporter is not there
For top box volume exporter, incoming line is joining with box. You can connect the line to middle of the volume exported box. Do the same for other volume exported boxes.


⚠ Processors has degraded performance

In a8c301b, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 69 ms

Resolved: Processors performance has improved in a69f36e.


test: clickhouse integration test

Based on #2804

Incidentally, the test fails at this time (for some reason the signs seem to be opposite).
I have not yet identified what is causing this.

Close as conflicts appear to be unfixable.


Simplified code a bit

Use |= and drop unnecessary else-scope.

Codecov Report

Merging #132 into master will increase coverage by 0.04%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #132      +/-   ##
==========================================
+ Coverage   23.62%   23.66%   +0.04%
==========================================
  Files          13       13
  Lines        1092     1090       -2
==========================================
  Hits          258      258
+ Misses        821      819       -2
  Partials       13       13

Impacted Files
Coverage Δ

pkg/utils/utils.go
42.16% <0%> (ø)
:arrow_up:

pkg/commands/container_stats.go
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 857053d...78d7fda. Read the comment docs.

Thanks again.

No problem, thanks for merging!


Daemon to handle a queue to make database insertions

Not urgent but it might make sense to have a queue responsible for database queries (or at least just those that cause insertions) to prevent application code having to deal with locked tables.

Just move away from sqlite tbh #trash


Removed the SECOND validation for external volume names

Summary:
86475dd (MARATHON-8681) was about to relax the external volume name validation in order to allow passing options via the volume name, which is required for some drivers to function as expected. However, an additional validation was missed, which is still in place. This issue is about removing that extra validation in order to fully support e.g. Encrypted Volumes with Portworx and UCR.
JIRA issues: MARATHON-8697

@jeschkies I adjusted this PRs description as well as the JIRA


use relative imports

I generally prefer to use explicit relative imports rather than absolute imports for anything within the same package. I looked into this in more detail, and at this point neither option seems to be strongly recommended over the other, so I suppose this is mostly a stylistic choice. I like the relative imports because you can import a package or module from anywhere without having to install it or mess with sys.path (this was the issue that led me to look at this). They also mean you can change the name of the package without modifying the files within the package.

You feel pretty strongly about this. :-) I'm happy to make the change! But can you sync with the master branch and then re-push? That should (I hope) get past the errors in the test.

Codecov Report

Merging #23 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master      #23   +/-   ##
=======================================
  Coverage   93.01%   93.01%
=======================================
  Files           9        9
  Lines         415      415
  Branches       65       65
=======================================
  Hits          386      386
  Misses         17       17
  Partials       12       12

Flag
Coverage Δ

#unittests
93.01% <100%> (ø)
:arrow_up:

Impacted Files
Coverage Δ

func_adl/EventDataset.py
94.28% <100%> (ø)
:arrow_up:

func_adl/util_ast_LINQ.py
100% <100%> (ø)
:arrow_up:

func_adl/ObjectStream.py
83.67% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 22075e1...9883f54. Read the comment docs.


Automation Fires Twice in 0.72.1

Hi,
I noticed that after upgrading to 0.72 (and then 0.72.1) at least some of my automations fire twice. I'll post a simple example of SMTP notification (which sends a text to my phone via AT&T actually) and the text/email now comes in twice on both open & close actions. Please see my config and auto yaml examples below. As you'd imagine, before 0.72 this did not happen. I believe I came from 0.71 but it could have been 0.69.x. I keep up with updates for my own sanity so my jumps are short when it comes to versions. This seems to have been recently introduced.
Thank you!
UPDATE: I have another example (if needed) of calling IFTTT for a siren trigger upon door sensor opening. IFTTT also shows that it was executed twice in the app and IFTTT online logs. I also verified that MQTT did not send the open message twice to trigger it twice. Definitely seeing a single MQTT message coming in.
Config:
notify:
  - platform: smtp
    name: sms_notification
    server: mx3a.txt.att.net
    port: 25
    sender: !secret sms_email
    recipient: !secret sms_email

Automations:
#########################################################      GARAGE OPEN & CLOSE NOTIFICATION
- alias: Garage Open Notification
  id: garage_open_notification
  trigger:
    entity_id: binary_sensor.garage_sensor
    platform: state
    from: 'off'
    to: 'on'
  action:
    - service: notify.sms_notification
      data:
        message: "Garage just opened."

- alias: Garage Closed Notification
  id: garage_closed_notification
  trigger:
    entity_id: binary_sensor.garage_sensor
    platform: state
    from: 'on'
    to: 'off'
  action:
    - service: notify.sms_notification
      data:
        message: "Garage just closed."

Please follow the issue template.
Use ps aux | grep hass check how many home assistant instance are you running.

That's a great suggestion, should of thought of that as well... I'll also reboot right now and see how it goes. However, I'm definitely running a single copy of the hass binary. See below, and apologies for how many times hass shows up because my username on that box is also hass (not the smartest move in hindsight). See below:

Apologies for not following the template... I was hoping not to inundate and get to the point but I'll comply below:

Running latest, v0.72.1
Not a custom component but an automation
This is a bug IMO
Pasted all pertinent details in my posts and supplementing with the hass log below.

2018-07-04 12:33:18 WARNING (MainThread) [homeassistant.components.binary_sensor.mqtt] No matching payload found for entity: Garage Switch State with state_topic: smartthings/Garage Switch/switch
2018-07-04 12:33:18 WARNING (MainThread) [homeassistant.components.binary_sensor.mqtt] No matching payload found for entity: Garage Switch State with state_topic: smartthings/Garage Switch/switch
2018-07-04 12:33:19 WARNING (MainThread) [homeassistant.components.binary_sensor.mqtt] No matching payload found for entity: Garage Switch State with state_topic: smartthings/Garage Switch/switch

Stopped doing it after upgrading to 0.76.2 --- now only receiving single action in IFTTT/email/text


Merge Zipkin v3 to the main branch.

We are planning to run a 3.0.0-rc0 release as we have finished all features covered in v3.\
The key features are as follows

Newly support for all versions of Elasticsearch 8 and OpenSearch 1.x. And keep supports for Elasticsearch 6 and 7.
Rapid support for the latest JDK releases, and fix for CVEs of dependencies.
Refactored storage structures and query logic for Elasticsearch to improve performance. Elasticsearch. Reduce 20% disk volume and 40% query latency on the 2 million spans case.
SQL Databases(MySQL and PostgreSQL) storage options are ready and stable for the product ready in the medium-scale deployment, or larger-scale when picking SQL Database mutations, such as Amazon Aurora.
Introduce cluster mode deployment in Zipkin v3. It reduces the memory requirements for the caches of service names, tags, and local/remote endpoints. It doesn’t require an extra deployment of Elastic Curator for Zipkin data time-to-live(TTL) in the Elasticsearch storage. SQL Databases(MySQL and PostgreSQL with their mutations) storage options support TTL for the first time.
Support for Apache BanyanDB (Alpha stage) as a new storage option, which is expected to offer better compression rates, lesser memory usage, and less CPU consumption compared to Elasticsearch data capacity(large-scale deployment) scenarios.
Automatic conversion of OpenTelemetry trace format into Zipkin via gRPC and storing it as Zipkin native spans, eliminating the need for an OpenTelemetry Collector relay.
A more powerful standalone memory mode, replacing the previous simple memory structure with H2 database, and supporting H2 DB Server for lightweight deployment.
Enable self-observability of the Zipkin server, trace querying, and span reporting by binding them to different HTTP IP addresses and ports. It provides more flexibility in a complex and security-sensitive environment.

Changes look OK to me but I will wait for others to review.
The only thing that catched my eye strongly was the env vars renaming. While this makes sense in some cases that I could identify, we must make sure migration isn't that much of a pain to users (that will have to drop their existing data) and make sure old zipkin config remains working. One of the most important principles in entire zipkin ecosystem was to not to break sites even on major version changes, instead deprecate them in a major and remove them in the next major.

So let's not rush to land this in main branch. I'd rather do release candidates with some real effort of reaching out to community and site owners (including a serious mention about this on our website).

So, is it fine to release one from a branch? If this is OK, we can work on that way.

So let's not rush to land this in main branch. I'd rather do release candidates with some real effort of reaching out to community and site owners (including a serious mention about this on our website).

So, is it fine to release one from a branch? If this is OK, we can work on that way.

IMHO it is fine to do release candidates / tech previews from a branch. Not sure how much work it will be to make it available. I just don't feel it's right to merge to main branch prior to the broader community approval.
I want this effort to be correct and supported by the broader community. The impact is too large to swiftly go through without giving more time and consideration.

IMHO it is fine to do release candidates / tech previews from a branch. Not sure how much work it will be to make it available.

OK, let me do this. (Release RCs from the branch)

Once you can release an image we could test the automation in
https://github.com/openzipkin/zipkin-php-example as smoke test first.
José Carlos Chávez
ons. 15. nov. 2023 kl. 10:45 skrev 吴晟 Wu Sheng @.***>:

Closed #3594 https://github.com/openzipkin/zipkin/pull/3594.
—
Reply to this email directly, view it on GitHub
https://github.com/openzipkin/zipkin/pull/3594#event-10965184143, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXOYAUZOWYXLVMZGJ6AR7DYESFLPAVCNFSM6AAAAAA7MEH5RSVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJQHE3DKMJYGQYTIMY
.
You are receiving this because your review was requested.Message ID:
@.***>


[Pylance] BaseModel is allowing any extra field

Checks

[x] I added a descriptive title to this issue
[x] I have searched (google, github) for similar issues and couldn't find anything
[x] I have read and followed the docs and still think this is a bug

Bug
class A(BaseModel):
    property1: str

a = A(property1="a")
a.property1 = "b"
a.anything = "shouldfail"

Unfortunately the a.anything = "shouldfail" line is not triggering a typing issue. And I don't think there is any way to do it.
Context
Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
pydantic version: 1.9.0
            pydantic compiled: True
                 install path: /Users/theo/Library/Caches/pypoetry/virtualenvs/platforms-K7nYilEZ-py3.8/lib/python3.8/site-packages/pydantic
               python version: 3.8.12 (default, Feb 23 2022, 18:58:53)  [Clang 13.0.0 (clang-13<IP_ADDRESS>)]
                     platform: macOS-12.2-arm64-arm-64bit
     optional deps. installed: ['typing-extensions'

This is a limitation / feature of the dataclass_transform mechanism that pydantic (and other similar libraries) use to allow type checkers to recognize models as dataclass-ish. There's currently no method by which a dataclass can specify that additional fields shouldn't be allowed.
Because dataclasses themselves don't allow this, I wouldn't expect it to be possible:
from dataclasses import dataclass

@dataclass
class MyDataclass:
    a: str

dc = MyModel(a="hello")
dc.b = "does not error either"

If you'd like this to exist, I would recommend leaving a message in the Github discussion in microsoft/pyright where it was originally proposed.

This actually works with dataclasses.

🤦🏽  My bad - my lower snippet was still using MyModel. That's interesting. I do still think it's as-designed for the dataclass_transform spec, but I can see a compelling reason for it to account for this case.

We’re no longer actively developing Pydantic V1 (although it will continue to receive security fixes for the next year or so), so we’re closing issues solely related to V1. If you really think this shouldn’t be closed, please comment or create a new issue 🚀.


Wont do sell order, showing "2019-07-28 15:15:41 (INFO):     NOT buying, already exposed"

Note: this is the technical bug tracker, please use other platforms for getting support and starting a (non technical) discussion. See the getting help page for details.
I'm submitting a ...
[ ] bug report
[ xx] question about the decisions made in the repository
Action taken (what you did)
I am using a buysell.js strategy
Expected result (what you hoped would happen)
If exposure is >10% and sell price is below candle.close , it should initiate sell order.
Actual result (unexpected outcome)
The logs register "INFO 2019-07-28 15:15:41 (INFO):     NOT buying, already exposed..," then it continues to collect data from gdax showing how many trades are happening , but wont initiate the sell order. Any thoughts why this is happening, I, for the life of me cannot figure out why it wont go to the ELSE IF "sell" loop after it realizes it is exposed.
The "buy side" works perfectly fine every single time. It is only the "sell" function that is not registering.
Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, etc)

Anybody having the same issue, this is so frustrating since the code all looks to check out :(

I'm just trying to find out why am I mentioned here :D I don't remember any code I published to this repo modifying buy/sell logic (I'm not even found the file you mentioned.) I think I only found a bug in the pg connector, and add a better slack integration. Honestly no idea what the problem is with this limited information.

Same issue, and it is really frustrating. Using neuralnet v2, 240 minutes history, 1 minute candle:
2019-09-29 21:14:36 (DEBUG):    Price = 7938.73, Prediction = 7973.848744331102
2019-09-29 21:15:36 (DEBUG):    Price = 7935.9, Prediction = 7972.802950769497
2019-09-29 21:15:36 (INFO):     NOT buying, already exposed
2019-09-29 21:16:36 (DEBUG):    Price = 7937.45, Prediction = 7971.737467114364
2019-09-29 21:17:36 (DEBUG):    Price = 7938.71, Prediction = 7970.795486360737


Prefer require_relative over require File.expand_path

They are the same: http://stackoverflow.com/a/26565471/1511504
But require_relative is easier to read.

I don't have a habit of using require_relative (mostly, I just don't think to use it), but it does seem to be a better option here.

Thanks. It looked a straight swap, but I was interested in what you thought of require_relative as I don't find Ruby file code intuitive.


Need more detail on Two-factor authentication when Clone

More information is needed on how to clone with token and Two-factor authentication.
Simply input the username + password or username + token, won't work.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 606d4c70-3daf-8b03-9c31-b690927a7136
Version Independent ID: 24a13aaf-3d14-c810-182c-2c2d93fc299b
Content: Setting up a Git Repository - Visual Studio for Mac
Content Source: mac/set-up-git-repository.md
Product: visual-studio-mac
Technology: vs-ide-general
GitHub Login: @heiligerdankgesang
Microsoft Alias: dominicn

@hanslai -- Hans, thank you for your feedback.  Please consider these resources:

Ask a question or find an answer on the Microsoft Q&A page for Visual Studio.
Check the Visual Studio MSDN Forum.
Stack Overflow
Reddit

@heiligerdankgesang -- Dominic, please look into this issue.

Sorry for the pile on, but I have the same (or at least similar) issue with VS 2022 on masOS Sonoma.
GH cli is installed (along with GH Desktop just in case it helped with auth in some way).
When attempting to clone any repository from Github (for the first time, i.e. never having cloned anything from GH before this moment) I am prompted for Username/Password (which I'm to understand is meant to be username+access token) but after this is provided the window simple hangs (freezes/timeout) and I have to eventually force close the application finding nothing but the project directory to be created.
gh version 2.32.0 (2023-07-11)
Visual Studio 17.6.1 (Build 452)
git version 2.41.0
What is the proper way to do this?


Release 2.10

Version 2.10
============

Enhancements and new features:

    * New plugin to scan remote Web sites (URL) (issue #981)
    * Add trends in the Curses interface (issue #1077)
    * Add new repeat function to the action (issue #952)
    * Use -> and <- arrows keys to switch between processing sort (issue #1075)
    * Refactor __init__ and main scripts (issue #1050)
    * [WebUI] Improve WebUI for Windows 10 (issue #1052)

Bugs corrected:

    * StatsD export prefix option is ignored (issue #1074)
    * Some FS and LAN metrics fail to export correctly to StatsD (issue #1068)
    * Problem with non breaking space in file system name (issue #1065)
    * TypeError: string indices must be integers (Network plugin) (issue #1054)
    * No Offline status for timeouted ports? (issue #1084)
    * When exporting, uptime values loop after 1 day (issue #1092)

Installation:

  * Create a package.sh script to generate .DEB, .RPM and others... (issue #722)
  ==> https://github.com/nicolargo/glancesautopkg
* OSX: can't python setup.py install due to python 3.5 constraint (issue #1064)

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Added markdownlint icon

Maybe consider using my icon?
https://github.com/AtomMaterialUI/iconGenerator/blob/master/assets/icons/files/markdownlint.svg

Maybe consider using my icon?
https://github.com/AtomMaterialUI/iconGenerator/blob/master/assets/icons/files/markdownlint.svg

I also thought about this icon, but I have doubts about its design, it seems to me that the check mark is too big and too far to the right

Maybe consider using my icon?
https://github.com/AtomMaterialUI/iconGenerator/blob/master/assets/icons/files/markdownlint.svg

I also thought about this icon, but I have doubts about its design, it seems to me that the check mark is too big and too far to the right

Maybe consider using my icon?
https://github.com/AtomMaterialUI/iconGenerator/blob/master/assets/icons/files/markdownlint.svg

I also thought about this icon, but I have doubts about its design, it seems to me that the check mark is too big and too far to the right

Guess it was bad timing for me 😅 https://github.com/material-extensions/vscode-material-icon-theme/pull/2429

Guess it was bad timing for me 😅 #2429

Your icon is really better, but still the problem with the ticks...
Perhaps it is better to make only one tick?

Is the logo with the tick an official logo for the library? I coudln't find any logo at all in their github repo.
Would you consider something like the following?

Btw, I also created this one with just one tick

Btw, I also created this one with just one tick

This icon is really great, but you need to move the check mark a little to the right

Like that?

Like that?

Yes!


pythonPackages.zstd: init at <IP_ADDRESS>

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

@eadwu May I suggest a following change
diff --git a/pkgs/development/python-modules/zstd/default.nix b/pkgs/development/python-modules/zstd/default.nix
index 5d1485c2720..785dd9d3946 100644
--- a/pkgs/development/python-modules/zstd/default.nix
+++ b/pkgs/development/python-modules/zstd/default.nix
@@ -1,4 +1,4 @@
-{ stdenv, buildPythonPackage, fetchpatch, fetchFromGitHub }:
+{ stdenv, buildPythonPackage, fetchpatch, fetchFromGitHub, pkgconfig, pytest, zstd }:

 buildPythonPackage rec {
   pname = "zstd";
@@ -8,18 +8,33 @@ buildPythonPackage rec {
     owner = "sergey-dryabzhinsky";
     repo = "python-zstd";
     rev = "v${version}";
-    sha256 = "0sy1ccqsh3bilzs7f6qkgqq9r734kgp8410vyv2x5vvzx3azp45d";
-    fetchSubmodules = true;
+    sha256 = "08n1vz4zavas4cgzpdfcbpy33lnv39xxhq5mgj0zv3xi03ypc1rl";
   };

-  patches = [
-    # make sure tests directory is included
-    (fetchpatch {
-      url = "https://github.com/sergey-dryabzhinsky/python-zstd/commit/cbdfbe2477b43e2d527fa4ee5dc960134a051be4.patch";
-      sha256 = "18ws563nmvx5rnpm1lyzi2379rgbsc614wp2cd5m40k3j21g0sfq";
-    })
+  postPatch = ''
+    substituteInPlace setup.py \
+      --replace "/usr/bin/pkg-config" "${pkgconfig}/bin/pkg-config"
+  '';
+
+  nativeBuildInputs = [ pkgconfig ];
+  buildInputs = [ zstd ];
+
+  setupPyBuildFlags = [
+    "--external"
+    "--include-dirs=${zstd}/include"
+    "--libraries=zstd"
+    "--library-dirs=${zstd}/lib"
   ];

+  # Running tests via setup.py triggers an attempt to recompile with the vendored zstd
+  ZSTD_EXTERNAL = 1;
+  VERSION = zstd.version;
+  PKG_VERSION = version;
+  checkInputs = [ pytest ];
+  checkPhase = ''
+    pytest
+  '';
+
   meta = with stdenv.lib; {
     description = "Simple python bindings to Yann Collet ZSTD compression library";
     homepage = "https://github.com/sergey-dryabzhinsky/python-zstd";
diff --git a/pkgs/top-level/python-packages.nix b/pkgs/top-level/python-packages.nix
index 2e99065a727..b2554dc53bf 100644
--- a/pkgs/top-level/python-packages.nix
+++ b/pkgs/top-level/python-packages.nix
@@ -10917,7 +10917,9 @@ EOF

   todoist = callPackage ../development/python-modules/todoist { };

-  zstd = callPackage ../development/python-modules/zstd { };
+  zstd = callPackage ../development/python-modules/zstd {
+    inherit (pkgs) pkgconfig zstd;
+  };

   zxcvbn-python = callPackage ../development/python-modules/zxcvbn-python { };

You don't need the https://github.com/sergey-dryabzhinsky/python-zstd/commit/cbdfbe2477b43e2d527fa4ee5dc960134a051be4.patch . You switched to fetching from github and that was enough to get the tests directory, it is just not available from Pypi.

@GrahamcOfBorg build python2Packages.zstd python3Packages.zstd

Success on aarch64-linux (full log)
Attempted: python2Packages.zstd, python3Packages.zstd
Partial log (click to expand)
rootdir: /build/source, inifile:
collected 16 items

tests/test_compress.py ........s..ss                                     [ 81%]
tests/test_version.py s..                                                [100%]

===================== 12 passed, 4 skipped in 0.08 seconds =====================
pytestcachePhase
/nix/store/0n0q3qal28h8nx6vd5jxydqiphk88ax3-python2.7-zstd-<IP_ADDRESS>
/nix/store/1l02m8hpn43gigbj4jakszc6fcf9qvj4-python3.6-zstd-<IP_ADDRESS>

Success on x86_64-linux (full log)
Attempted: python2Packages.zstd, python3Packages.zstd
Partial log (click to expand)
rootdir: /build/source, inifile:
collected 16 items

tests/test_compress.py ........s..ss                                     [ 81%]
tests/test_version.py s..                                                [100%]

===================== 12 passed, 4 skipped in 0.07 seconds =====================
pytestcachePhase
/nix/store/8i3ry1bylrrp2ny82blq3dq2cjlxqvya-python2.7-zstd-<IP_ADDRESS>
/nix/store/dzxsqlvpji5n18g1g8vq8wmdj1q6gylh-python3.6-zstd-<IP_ADDRESS>

Success on x86_64-darwin (full log)
Attempted: python2Packages.zstd, python3Packages.zstd
Partial log (click to expand)
collected 16 items

pytestcachePhase
tests/test_compress.py ........s..ss                                     [ 81%]
tests/test_version.py s..                                                [100%]

===================== 12 passed, 4 skipped in 0.11 seconds =====================
pytestcachePhase
/nix/store/y4mwby2lkva9i1bxwi9y6cj4j33bysr9-python2.7-zstd-<IP_ADDRESS>
/nix/store/gap9fwnky5za6z7kym7blkx3x8ykkhkf-python3.6-zstd-<IP_ADDRESS>


Some documentation pages are appearing empty when they shouldn't be

From @chrisirhc:
This weird state where the left-hand panel seems to be affecting other modules/versions randomly:

<EMAIL_ADDRESS>(Ok)
<EMAIL_ADDRESS>(Bad)
<EMAIL_ADDRESS>(Ok)

For me they all look the same; what exactly is the issue?

Details were in https://github.com/denoland/dotland/issues/2472

Looks like this is no longer reproducible on the link I shared. Feel free to close. Perhaps another example will come up.

Ah, actually is was broken for me, then I did a reload manually, but it didn't fix it because I forgot the api isolate caches the response, so it fixed it.
We need to have a cleanup job that runs every hour that mops up failed loads because of CPU time limits.

The cause of this is that it's caching the blank page that's rendered due to CPU time limits?
Is the caching behavior part of Deno Deploy, Fresh, or the code in dotland/docland?

This seems to be fixed.


separate moment-timezone from the main bundle

moment.js was added to our lib to convert UTC date-times into Local timezone date-times in the Explorer's Table view.
/lib/libraries/default/table:63
moment.js is a pretty heavy library, and most of the devs use the API timezone parameter anyway, so let's remove it from the keen-dataviz.js
in the Explorer, table values will be converted using
https://github.com/keen/keen-dataviz.js#mapping-values-of-a-table-column

#187


Fix typo in vectors.cpp

poniting -> pointing

And int_vector.begin() + 2 should be the third element.

And int_vector.begin() + 2 should be the third element.

feel free to submit a pull request :)


Add URI filtering

This PR adds URI filtering to outgoing webhooks. We don't want users to be able to make webhook requests to our application or other internal resources that our machine may have access to. We filter both as a model validation and when attempting delivery. We do both because a validation gives immediate feedback to users but can't be trusted because a user could enter a valid hostname, change it to a private IP, and then make a webhook request.
This PR is heavily inspired by Smokescreen, a filtering egress proxy written in Go. We use the same ideas as smokescreen and apply them as a Rails concern with a small modification to DNS resolution. Smokescreen assumes an upstream recursive resolver (i.e. running unbound as a sidecar) whereas we can't assume that for applications using Bullettrain. Instead we do a partially recursive resolution, where we ask an upstream server for the NS record for the domain and then ask the authoritative server directly for the A record.

@petekeen-cf Really sorry for the inconvenience here, but can you push this PR up again https://github.com/bullet-train-co/bullet_train-outgoing_webhooks ?

Done https://github.com/bullet-train-co/bullet_train-outgoing_webhooks/pull/1#pullrequestreview-884679883


Describe the bug
I can't find the publicNetworkAccess parameter in the code at the path ResourceModules/modules/document-db/database-account/main.bicep.
In the old modules, the publicNetworkAccess parameter was present. This is crucial for us to disable public access and manage it via private endpoints.
To reproduce
After deployement you found that the network Access enabled for public
Code snippet
No response
Relevant log output
No response

Hey @OussamaLarfaoui,
the module has been ported to AVM a while back. You can find the latest version of the module here.
In it, you will find that you can set the publicNetworkAccess property via the networkRestrictions parameter :)
I'll close this issue for this repository. If the latest version in AVM does not work for you, please create an issue over there. The module's owner would then help you directly :)

Thank you @AlexanderSehr  !!


Integrate Cert-Manager with AKV2K8S for Key Vault Azure

Hi,
I am trying to integrate Cert'manager with AKV2K8S.
I have this
apiVersion: v1
kind: Secret
metadata:
  name: ca-key-pair
data:
  ca.crt: xxx
  tls.key: xxx
  tls.crt: xxx

so I would like to use AKV2K8S to avoid hard coding the ca.crt, tls.key and tls.crt info.
apiVersion: cert-manager.io/v1
kind: ClusterIssuer
metadata:
  name: ca-issuer
  namespace: cert-manager
spec:
  ca:
    secretName: ca-key-pair

In Key Vault in Azure I have created 3 entries in Secrets for ca.crt. tls.key and tls.crt.

Any idea how can I integrate the AKV2K8S so I can specify the kv2k8s secrets into the ClusterIssuer?
Thanks in advance.

Hello @JimFran, Did you manage to integrate Azure Key Vault using cert-manager?

Hi,
you can create three AzureKeyVaultSecrets with the same output secret name and akv2k8s will combine them into a single secret:
https://akv2k8s.io/tutorials/sync/5-multi-akvs-to-one-secret/


Error when running in git repo

I get the following error when I try to run it in a git repo:
Error: Current directory is already an exisiting git repository!

README.md and LICENSE are not created yet.

Added in 1.1.0:
https://github.com/frdmn/init.js/blob/master/CHANGELOG.md


uninstall

how does one uninstall this app?

Quit the Kite Menubar
Drag Kite.app to the Trash
(If you installed Kite from Atom then uninstall the Kite package in Atom.)


FP for "case" in Rule 942230

Description
The following trigger false positives.
Just in case (abc)
In one case (example 1)

Audit Logs / Triggered Rule Numbers
ModSecurity: Warning. Matched "Operator `Rx' with parameter `(?i:[\s()]case\s*?\(|\)\s*?like\s*?\(|having\s*?[^\s]+\s*?[^\w\s]|if\s?\([\d\w]\s*?[=<>~])' against variable `ARGS:test' (Value: `Just (15 characters omitted)' ) [file "/etc/nginx/modsec/rules/REQUEST-942-APPLICATION-ATTACK-SQLI.conf"] [line "218"] [id "942230"] [rev ""] [msg "Detects conditional SQL injection attempts"] [data "Matched Data:  case ( found within ARGS:test"] [severity "2"] [ver "OWASP_CRS/3.3.0"] [maturity "0"] [accuracy "0"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-sqli"] [tag "paranoia-level/1"] [tag "OWASP_CRS"] [tag "capec/1000/152/248/66"] [tag "PCI/6.5.2"] [hostname "<IP_ADDRESS>"] [uri "/"] [unique_id "16033842602.058623"] [ref "o7,7v11,19t:urlDecodeUni"]

Your Environment
CRS version: v3.3.0 with tweaks
Paranoia level setting: 1
ModSecurity version: 3.0.4 with minor truncating match tweak.
Nginx
Amazon Linux 2
Confirmation
[X] I have removed any personal data (email addresses, IP addresses,
passwords, domain names) from any logs posted.
Possible Solution
Is there an SQL language that supports CASE (?  I thought it was CASE ... WHEN ... THEN ... ELSE ... END.  Also, maybe a SELECT can be required too like #1806, so maybe this rule would work:
- SecRule REQUEST_COOKIES|!REQUEST_COOKIES:/__utm/|REQUEST_COOKIES_NAMES|ARGS_NAMES|ARGS|XML:/* "@rx (?i:[\s()]case\s*?\(|\)\s*?like\s*?\(|select.*?having\s*?[^\s]+\s*?[^\w\s]|if\s?\([\d\w]\s*?[=<>~])" \
+ SecRule REQUEST_COOKIES|!REQUEST_COOKIES:/__utm/|REQUEST_COOKIES_NAMES|ARGS_NAMES|ARGS|XML:/* "@rx (?i:select.*[\s()]case\s+.*\s+when\s+.*\s+then\s+.*\s+end|\)\s*?like\s*?\(|select.*?having\s*?[^\s]+\s*?[^\w\s]|if\s?\([\d\w]\s*?[=<>~])" \

Closing this in favor of PR #2035.


GCNConv error?

Not sure how else to explain this error but to post it here. I am using GCNConv but am running into this error randomly at higher epoch count:
Run: 01, Epoch: 316, Loss: 61.6280, Valid MSE: 215.70%, Test MSE: 305.34%
100%|██████████| 2301/2301 [01:32<00:00, 24.98it/s]
100%|██████████| 203/203 [00:07<00:00, 25.86it/s]
100%|██████████| 204/204 [00:08<00:00, 25.34it/s]
MSE
Run: 01, Epoch: 317, Loss: 63.0244, Valid MSE: 215.28%, Test MSE: 310.71%
100%|██████████| 2301/2301 [01:31<00:00, 25.15it/s]
100%|██████████| 203/203 [00:07<00:00, 25.45it/s]
100%|██████████| 204/204 [00:07<00:00, 25.72it/s]
MSE
Run: 01, Epoch: 318, Loss: 63.0782, Valid MSE: 220.16%, Test MSE: 312.79%
100%|██████████| 2301/2301 [01:33<00:00, 24.58it/s]
100%|██████████| 203/203 [00:07<00:00, 26.03it/s]
100%|██████████| 204/204 [00:08<00:00, 24.93it/s]
MSE
Run: 01, Epoch: 319, Loss: 62.9604, Valid MSE: 211.49%, Test MSE: 312.12%
100%|██████████| 2301/2301 [01:35<00:00, 24.19it/s]
100%|██████████| 203/203 [00:07<00:00, 27.58it/s]
100%|██████████| 204/204 [00:07<00:00, 26.32it/s]
MSE
Run: 01, Epoch: 320, Loss: 61.7545, Valid MSE: 212.12%, Test MSE: 298.78%
100%|██████████| 2301/2301 [01:31<00:00, 25.05it/s]
100%|██████████| 203/203 [00:07<00:00, 27.63it/s]
100%|██████████| 204/204 [00:07<00:00, 26.85it/s]
MSE
Run: 01, Epoch: 321, Loss: 62.5692, Valid MSE: 214.07%, Test MSE: 301.77%
100%|█████████▉| 2300/2301 [01:33<00:00, 24.54it/s]
Traceback (most recent call last):
  File "/path/to/run4.py", line 476, in <module>
    loss = train_model()
  File "/path/to/run4.py", line 69, in train_model
    else: out = model(data).view(-1)
  File "/home/vym1/nn2/lib/python3.7/site-packages/torch/nn/modules/module.py", line 727, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/path/to/GNN.py", line 107, in forward
    xs += [torch.tanh(GCNConv(xs[-1], edge_index))]
  File "/home/vym1/nn2/lib/python3.7/site-packages/torch/nn/modules/module.py", line 727, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/home/vym1/nn2/lib/python3.7/site-packages/torch_geometric/nn/conv/gcn_conv.py", line 161, in forward
    self.improved, self.add_self_loops, dtype=x.dtype)
  File "/home/vym1/nn2/lib/python3.7/site-packages/torch_geometric/nn/conv/gcn_conv.py", line 52, in gcn_norm
    edge_weight = torch.ones((edge_index.size(1), ), dtype=dtype,
IndexError: Dimension out of range (expected to be in range of [-1, 0], but got 1)

This seems random and I'm not sure how to fix it? The model worked fine for the first 300 or so epochs.

Can you show me the shape of edge_index for which this error occurs?


Proposal: Sections

Describe the feature you'd like
I've been reading feature requests for things like "sub-chapters" and the like. This concept is similar, but involves a layer above shelves.
Carrying forward the library metaphor used in Bookstack ... libraries do have shelves of books, but those shelves are arranged in "sections". Otherwise, it would take you forever to find what you were looking for.

History Section

Shelf A - American History

The Revolutionary War
History of American Presidents
JFK and Elvis: The Real Truth

Shelf B - European History

Charlemagne and his Lady Friends
Hitler and his Secret Obsession with Egg Coddlers

Shelf C - World History

What are Pymgies Anyway?
The Forgotten Pirates of the Antarctic

Fiction Section
Poetry Section
Children's Section
etc.

For a more realistic Bookstack-centric example, we currently have something like the following (with one shelf expanded a little here):

Shelf: Client Implementations
Shelf: Marketing
Shelf: Sales
Shelf: Technical

Book: Ansible Implementation
Book: Coding Standards
Book: System Administration
etc.

The books, of course, have chapters and pages in them. Now, the "Ansible Implementation" book (along with a lot of others) really should fall under some sort of "System Administration" category (currently would be a "shelf"). However, I don't want to have more than one shelf related to technology - I just want one shelf to rule them all.
What I'd like to be able to do, is something like this:

Section: Client Implementations

Shelf: Premier Accounts

Book: Acme Corporation
Book: Frobozz Inc.
etc.

Shelf B
Shelf C

Section: Technical

Shelf: Coding Standards

Book: PHP/Laravel
Book: Shell Scripts
etc.

Shelf: System Administration

Book: Ansible Implementation
Book: Disaster Recovery
Book: Scheduled Jobs

Chapter: Daily Jobs
Chapter: Hourly Jobs
Chapter: Laravel Jobs
etc.

Benefits of the Feature
These are probably somewhat obvious, but:

Directly provides a way to group all related content under a single umbrella at the top level
Enhances the current "library" metaphor
More ways to organize things in any data-centric application is generally better

Additional context
Currently, the app uses separate models/tables for bookshelves, books, chapters, and pages. Obviously, the easiest thing to do would be to add another model for Section that would parent shelves. However, a more robust approach would be to use the most excellent calebporzio/parental package to have a base BookstackContainer class (or whatever you wanted to call it), which could easily be extended by other classes (even user-defined ones) to provide whatever level of organization the user wanted.
Just a thought :)

After reviewing the code base, it would appear that adding a "Section" layer would be by far the easiest thing to do. Making it definable would require a complete overhaul of the application as quite a bit of functionality is tightly coupled with each of the existing layers.

I've been thinking about this a bit more. I've forked off the current master and soon may take on the effort I've described in the Additional Context section above. Would love anyone interested in having that capability react with a thumbs up or what not.

Hi @sburkett,
Thank you for the detailed thought-out suggestion.
Personally I'd prefer not to support another level in the hierarchy. My comment from 2016 here still represents how I'd feel about this; If anything I'm probably more strongly against this now than in 2016 since we now have shelves.

Sure, no problem. I do feel that a completely configurable hierarchy with a set of defaults of what is there now (shelf->book->etc) is the ideal scenario for everyone, though. No doubt it is a tremendous undertaking, since it almost becomes a rewrite, but I might take it on just for kicks lol. We'll see. Life will most assuredly get in the way!


Defer to core.rollMode setting when assigning most roll modes

Since when did "roll" grab the core rollmode? That's neat, wonder how I missed that one.


Bug when moving list items up and down

On latest main, using the following file:
- A
  - B
- C
  - D
    - E

With the cursor on the E, doing (lists-moveup) followed by (lists-movedown) puts the E in a different place from where it started (I guess these ops should be inverses), and another (lists-movedown) causes an error to fire in the plugin.

You raise a valid issue. I spent some time thinking about these things. What do we want to happen if we move E up through the list? I think there are several choices here, because we need to choose how to treat the list level. In the current implementation, the idea is that moving list items do not change the level. Thus, moving E up should move it to A -> B -> E. This seems to work. Moving it down should move it back to the original spot, and I agree this seems to not work as expected.
The error occurs for the same reason, because the E entry now hangs without a close parent (it has level 3, whereas the parent has level 1). The code does not handle this error properly - I'll fix that as well.

I agree that moving vertically should not change the level, and if I were to guess, I would say that the way this should work is the same as how the plugin works for this snippet, which for some reason is different from the one I posted (which may itself be a different bug):
- A
  * B
- C
  * D
    + E

(It moves both D and E up/down.)

I believe the original issue (both parts) is fixed now, please test.

The new snippet uses a new marker + which is not recognized as a list bullet. Thus, the text is considered a continuation of the item.
The core idea is more or less to use the indentation to determine the list items and list structure.

Just tested, LGTM. It looks like when moving a node, you jump to the nearest available slot at the node's level in the given direction, which I think makes sense and has the benefit of being easy to understand. Thanks again for the plugin!

Great; thanks for using this and reporting the issues! Glad to hear it works as expected for you.
I've updated the docs to clarify the types of lists supported. I hope that may make it clearer to other new users.


Initial implementation of lint subcommand

This PR initially implements brand-new lint subcommand as we discussed: https://github.com/wolfi-dev/os/issues/86

Implement bare-minimum lint subcommand
Add 3 rules: no-makefile-entry-for-package, valid-copyright-header and valid-pipeline-fetch-uri (for test purposes)
Ran gofumpt -w -extra . across project (some intentional changes has made)
Move some Melange-related funcs to melange.go for code deduplication
No tests written yet, we thought we can do once we get agreement on the overall architecture
Bumped the Melange to latest version, so it brings some new dependencies.

Future ideas:

Create a new /rules folder to separate rules (as godolint did)
Give ID for each rule? (i.e., WL0001)
Better output format (table output, nicer log printing)
Verbose mode to print some additional description and stuff
Rich options (fail-fast, exclude/include rules by IDs, override exit code by severity, etc.)

Known Bug?:

We couldn't add strict field checking while unmarshaling given YAML since secfixes didn't introduce in build.Configuration yet on main branch but generally used in the package files.

Example Output:
$ go run . lint os -v
found 198 packages
2022/12/15 12:37:23 Package: wait-for-it: 1 error occurred:
	* [no-makefile-entry-for-package]: package wait-for-it is not exist in the Makefile (ERROR) - (every package should have a corresponding entry in Makefile)

(Exit code is 0, should we return 1? (if severity >= ERROR))
/cc @rawlingsj @developer-guy
Signed-off-by: Furkan<EMAIL_ADDRESS>Co-authored-by<EMAIL_ADDRESS>

this looks great!
I think we can merge on the agreement of a followup PR to add tests if that's ok with you?

(Exit code is 0, should we return 1? (if severity >= ERROR))

Yeah let's return 1 so that we can easily fail CI?

Or if you prefer to add a test to this PR that's probably better and good practice.  To answer the initial question I think this is a great start and we can iterate over time to improve.

Makes sense! We will add some test cases in this PR. (+ exit 1)

perfect - one more thing you might want to do is run make docs to generate the CLI docs.

We also thought enabling/disabling options for the available linting rules might be a good option, WDYT?

We also thought enabling/disabling options for the available linting rules might be a good option, WDYT?

yeah that sounds good though maybe as a followup PR?

I think we are ready for the review! @rawlingsj

Add some test cases (last commit)
Separate digest check logic to new rule: valid-pipeline-fetch-digest
--list option to list all available rules

@Dentrax @developer-guy great work!

I'll look at getting wolfi-os using the new linter and see how we go.

Thanks for such prompt action! Looking forward to the new rules! 🚀

Most of the development effort belongs to @Dentrax, I just added couple things, so thx @Dentrax 🏅


Generating all autoInc values from firstInsertId

This change is

Review status: 0 of 3 files reviewed at latest revision, 1 unresolved discussion.

java/jdbc/src/main/java/com/flipkart/vitess/jdbc/VitessStatement.java, line 379 at r1 (raw file):
Previously, sougou wrote…
I presume you're mimicking the mysql JDBC driver? Can you add a comment that says so?

Done.

Comments from Reviewable


Next is 2.11.0

@zhfeng Many thanks for handling the release. The first time could seems a bit challenging.
I think this PR reports errors because you would need to run:
mvn clean install -DskipTests

If I'm right, it would update some file that could be committed.


Domain can not be transferred

This script is really cool. I am coming from WHMCS and am very impressed.
So far I only seem to be having an issue with the ResellerClub domain module. I followed the steps (got my userid and registered IP's in their control panel) and when choosing domain transfer during the order process, I usually get "Domain can not be transferred" in the pop-up dialog. I occasionally get other messages when trying random domains like "ResellerClub: "rewererw.com is currently available for Registration. You may Register this domain name by placing a Registration request with us." This leads me to believe the module is doing something.
Any suggestions? Thank you for your time.

Have you enabled domain transfers for TLD in admin area?

Yes, transfer is enabled for all TLD's I set up in Box Billing.

Hi everyone. I try boxbilling and it's really cool.
I have also getting the same issue "Domain can not be transferred." when trying to transfer domain form other service provider with the transfer option available in boxbilling.
https://support-tools.com/?f8748dc9077701ac#3JWJDslcF/jPwvrPEFjPUndx7S+azX37WPRZM5Tdo0M=
https://support-tools.com/?5dc2c08b03cfaecd#G8a9a9/mv70Udhpcrd0uWxV68zNxyTyvyn4tSknkzDE=
Please help me how to solve it...
Thanks in advance...


Update pythonbuild.yml to fix the build

Description
PRs are failing due to github action python version updates from 3.8.9 to 3.8.10 for apache tests.
Same for 3.9.4 to 3.9.5.
Motivation and Context

Have you tested this? If so, how?

we should disable the 3.8 and 3.9 checks from the settings of the repo, in order to not expect them to run successfully and get this merged


Implementation of SneakingData

Implements SneakingData with new DataAPI - New fork, new branch.
I’m new to git. ._.

Please update according to the changes recently merged.

You guys take all the fun out if it. :smirk:

Please double check the requirements of implementing a DAtaProcessor by extending an AbstractSpongeDataProcessor, as well as writing a DataManipulatorBuilder.

Well, most of the fun.

 Please fix that. As well as the few other places you're not using the ImmutableDataCachingUtil.

Done -

Woops.

:+1:

Please update according to the recent changes for Entity specific DataProcessors.


JavaJDK -> OpenJDK

We now install OpenJDK, and should give directions to how to delete that directory.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 739a5418-f754-69ea-e825-6c201a5aa9d8
Version Independent ID: 95fad532-95e9-b97e-a832-785a9ea9a283
Content: Uninstalling Xamarin - Xamarin
Content Source: docs/cross-platform/get-started/installation/uninstalling-xamarin.md
Product: xamarin
GitHub Login: @asb3993
Microsoft Alias: amburns

Thanks for submitting an issue and we apologize for the delayed response. The requested updates haven't been made since the creation of the issue, and the timeline for resolution may vary based on resourcing, so the issue is being moved to an internal repo to better track it.


Add support for specifying the PathBase for all requests

As a follow up from https://github.com/aspnet/Hosting/issues/815, we want to allow specifying the path base as a host setting. Hosting should use this to call UsePathBase as the first middleware in the pipeline. IISIntegration does something similar with the APPL_PATH  env variable : https://github.com/aspnet/IISIntegration/blob/51554566532237ef76cb105ec5b3f7a5050b9ba4/src/Microsoft.AspNetCore.Server.IISIntegration/WebHostBuilderIISExtensions.cs#L19.

You want to be able to switch between WebListener ( or any host) and Kestrel  which share the same startup before startup is loaded..    Having If (KestralLoaded) in startup is pretty damn ugly .. and makes testing a pain  , i dont think there should be any Host ( including Kestrel)  specific stuff in Startup .

Using IStartupFilter is pretty ugly and having  2 methods for the 2 web hosts is pretty poor .   As I mentioned before other systems may do a lot of work before startup eg console hosts , hosted by SF etc etc .  Core and web listener supports this nicely  but Kestrel does not. eg Hosting is a ASP.Net concept in startup I think the port/url hosting needs to be done before  Startup ..  Sure Startup should have access to this but it should be set before.

I'm not sure what you're referring to but there will be a use path base on the IWebHostBuilder like the method suggests. I don't know what more you're asking for.

@davidfowl For testing this change, I believe I need to send a request to the application to verify that the HttpContext properly contains the PathBase.
Firstly, is there a better way to test this?
If this is correct, what type of test would this be? It seems to extend past the WebHosts tests because it requires that a request be sent to a server.

@jkotalik Whats the issue with sending a request?

I assume you'd just be calling UsePathBase. https://github.com/aspnet/HttpAbstractions/blob/87cd79d6fc54bb4abf07c1e380cd7a9498a78612/src/Microsoft.AspNetCore.Http.Abstractions/Extensions/UsePathBaseExtensions.cs#L21
You can mimic existing hosting and UsePathBase tests.

Maybe it's a good idea to also make the use of proxy forwarding HTTP headers configurable for the hosting environment. Because these headers are provided by the hosting environment and could be different per host. The application shouldn't need to care about what headers the proxy provides.
My applications currently draw the application path base from an environment variable provided by my own hosting environment and put it into UsePathBase, and they also call UseForwardedHeaders because the hosting environment works that way.
But I can't influence third-party applications. They might stop working in some hosting environments.

I just stumbled on this issue again; what is the actual goal of this? To move the configuration of the path base into the WebHostBuilder, to ensure that it is always done at the very beginning of the middleware pipeline? And so that people can configure it through the host builder configuration (e.g. using environment variables, command line arguments, etc.?)

@poke yes that was the intent

I see, I might have the chance to work on this then, seeing that this is not planned by someone else at this point.
Does the solution include any support for allowing to dynamically set the path base when sitting behaind a reverse proxy? In one of my applications, I currently use a custom X-Forwarded-Path HTTP header to dynamically configure the path base (along with the other X-Forwarded headers) to completely uncouple the configuration from the application, moving it into the reverse proxy configuration.
I don’t assume that we would want to move such a functionality into the webhost but keep it as custom middleware then, right (Just like the UseForwardedHeaders middleware)? In that case, we would have to make sure that the stuff in the webhost will not conflict here and that you could still overwrite the path base properly in user middleware.

X-Forwarded-Path would be an interesting add to UseForwardedHeaders, though it appears to be far less common.
The ask for Hosting is only for a static config based mapping for basic reverse proxy scenarios.

That said, I've seen some variants of this request in forums that UsePathBase does not cover. If the proxy trims or prepends the path then UsePathBase doesn't work, it relies on an unchanged path.
Examples:
A basic pass through:
Public Path: /foo/api/1
Proxy forwards: /foo/api/1
Fixup needed: UsePathBase(/foo), Path becomes: /api/1 and PathBase becomes: /foo
Variant 1: Proxy trims the path
Public Path: /foo/api/1
Proxy forwards /api/1
Fixup needed: Set PathBase to /foo, leave Path alone
Variant 2: Proxy prepends path
Public Path: /api/1
Proxy forwards /foo/api/1
Fixup needed: Remove /foo from Path, don't set PathBase
UsePathBase might need some overloads to handle these: Transfer, Set, Trim. I don't know how easily we could get those all into hosting config unless they were all separate parameters.

Just got burned by this, specifically proxy trimming the path. Sorry I have nothing more valuable to add other than feedback from the wild.


use local cached model for dynas ut test

Signed-off-by: chensuyue<EMAIL_ADDRESS>Type of Change
validation
Description

Use local cached model for dynas ut test, walk around for connection timeout issue.
Fix ITEX ut issue.

Expected Behavior & Potential Risk
CI pass
How has this PR been tested?
CI test
Dependency Change?
No

ITEX baseline job run with master so it's make sense to be failed.


socket.io security issue

Depending on the type of issue, please include the follwing information:

Node and NPM Version

v14.15.4
6.14.10

WebSSH2 release version
"version": "0.2.10-1",
"release": "standard-version"
"standard-version": "^7.0.0",
OS and Version of SSH server connecting to

Linux ib21b13 3.10.0-1160.2.2.el7.x86_64 #1 SMP Sat Oct 17 05:06:47 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
OpenSSH_7.4p1, OpenSSL 1.0.2k-fips  26 Jan 2017

Browser Version and OS

Information from brwoser's About... or a screenshot of the about screen.

Any log or messages from the WebSSH2 output

Issue:
Recently, socket io report a security issue, we need upgrade socket.io version to 2.4.0 or later
https://exchange.xforce.ibmcloud.com/vulnerabilities/195255

This is targeted for 0.4.0, you can check this out now in a840ec1

fixed in 0.4.0 release


librenms: 24.9.1 -> 24.10.1

Things done
Updates LibreNMS to 24.10.1 and adds me as a maintainer again (I originally packaged it for Nix, and we now use it in my company, so I can maintain it again).

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[X] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofborg test librenms
@ofborg build librenms

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/1016/187


BasicAuth doesnt work

After my last PR, i have upgrade the action to V3 but still getting 401 from Loki when trying to send logs from pipeline

are you using a local or cloud loki instance?

local

How does the action connect to the instance? If the request actually lands, Is the auth failure reflected in the Loki logs?

As you can see at the Loki logs, the username and password are empty ( the first 2 dashes ).
I tried to trigger the action with hardcoded string username and password and it still empty

@dormullor thanks for the confirmation 👍  let me try to reproduce this

@dormullor I have create a PR that updates sub dependencies and I can confirm logs are received correctly.
One thing I found during my process was, that if you do not need user and password it is best not to add them to the secrets as an empty variable is different than a non-existing one.

Perfect ! work like a charm
Thanks !

Perfect ! works like a charm
Thanks !


Merge r2m PR#326 - Add support for mandatory installation of OMI

What does this PR do? Please provide some context
Add support for installing the latest version of OMI package.
Where should the reviewer start?
A bug was found related to the crontab entry recently, and a hot fix, omi-1.4.1-1, was released.  The fix is also available in most recent release, omi-1.4.2-1.  This bug occurred on systems that don’t have Kerberos installed, and caused an error email to be generated every minute.
How can this be manually tested? (brief repro steps and corpnet-URL with change)
The updated utilities.sh library file and how it is integrated with the installation process via the install-tools function.
What are the relevant TFS items? (list id numbers)
source utilities.sh
install-omi
Definition of done:

[x] Title of the pull request is clear and informative
[x] Add pull request hyperlink to relevant TFS items
[ ] For large or complex change: schedule an in-person review session
[ ] This change has appropriate test coverage
[x] Get at least two approvals

Reminders DURING merge

If you're merging from a short-term (feature) branch into a long-term branch (like dev, release, or master) then "Squash and merge" to keep our history clean.
If merging from two longterm branches (like cherry picks from upstream, dev to release, etc) then "Create merge commit" to preserve individual commits.

Thanks


Github workflow and README fixes

Hi there back with more 👋
What is this?
Previously I would pull the repo locally and then deploy it on my local server, having the images be automatically made makes it easier for me to update my local deployment of GTAV-NativeDB.
So what I've made in this PR is a Github workflow that builds a docker image every time you push to the master branch.
I've setup the workflow to be completely independent of the repository owner, it should be ready to merge as-is without any changes. The package will be automatically linked to this repository, you can take a look at my repo to see how it looks.
I've also added a devcontainer configuration, however that was more for personal testing and I'm happy to take it out if you want.
Additions

GH Workflow file to build a docker image on the current repository
devcontainer config (I use this to not dirty my desktop with different node installations)

Fixes

Fixed README logo image not showing up
Fixed small typo in README

Setup
To get this to work there are a few steps that you'll have to take.

Create a personal access token with package:write and repo:full access (ticking package:write will automatically add repo:full)
Set an expiry date on your token, take something that you feel comfortable with.
Add the token to your repository secrets with PACKAGE_WRITE_TOKEN as name.
Merge into master and watch build here.
After the build is a completed a new package should become visible on your account, however it will be set to private visibility by default.
Go to your package settings and change the visibility to public.

Issues
I see you've added environment variables to set the application title, description, etc...
I will investigate some more into fixing this for docker builds, would you want separate images to be built for RDR3 and GTA5?

Very neat, I'll look into merging this soon.
As for:

I see you've added environment variables to set the application title, description, etc...
I will investigate some more into fixing this for docker builds, would you want separate images to be built for RDR3 and GTA5?

Considering that the environment variables are injected at build time, having two seperate docker images is probably the only option.

I'll fix that tomorrow evening.

Ok that's fixed, basically I parse the .env.example file. On thing to note however is that the action makes that assumption that whatever it commented is the RDR3 env variables and whatever isn't commented is for GTA V.
You can change the contents of the variables whenever as long as the file keeps the RDR3 variables commented and GTA V variables uncommented.
I also saw that the browserlist was outdated but I'm leaving that for you to update.

Ok that's fixed, basically I parse the .env.example file. One thing to note however is that the action makes that assumption that whatever it commented is the RDR3 env variables and whatever isn't commented is for GTA V.
You can change the contents of the variables whenever as long as the file keeps the RDR3 variables commented and GTA V variables uncommented.
I also saw that the browserlist was outdated but I'm leaving that for you to update.

Thanks! I'll try merging tomorrow

It's all up and running, thanks again 🙂. https://github.com/DottieDot/GTAV-NativeDB/actions/runs/5728071063