Dart 2 issue

Hello,
after updating to dart 2. I get the following error when using Reducer

error is happend type '(OnBoardingState, LicensesLoadedAction) => OnBoardingState' is not a subtype of type '(OnBoardingState, dynamic) => OnBoardingState'

My code looks like the following:
final onBoardingReducer = combineTypedReducers<OnBoardingState>([
  new ReducerBinding <OnBoardingState, LicensesLoadedAction>(_setLoadedLicenses),
]);

OnBoardingState _setLoadedLicenses(OnBoardingState onBoardingData, LicensesLoadedAction action) {
  return onBoardingData.copyWith(licenses: action.licenses);
}

If I set the type to dynamic it works but I would prefer to not have to case the action.
OnBoardingState _setLoadedSchools(OnBoardingState onBoardingData, dynamic action) {
  if(action is SchoolsLoadedAction) {
    return onBoardingData.copyWith(schools: action.schools);
  }
  return onBoardingData;
}

Any idea?

I get a similar issue when dispatching actions using my store:
I/flutter ( 3621): The following assertion was thrown while handling a gesture: I/flutter ( 3621): type '(AppState, ChangeViewAction) => AppState' is not a subtype of type '(AppState, dynamic) => I/flutter ( 3621): AppState'
I believe changes to the redux_dart library need to be made in order to bring it up to speed with Dart 2, i'll try to investigate further

Thanks for the reports! I'll try to find time soon to look into the issues :)

@DrivEddy Hrm, are you passing the onBoardingReducer directly to the store? This is a curious case. I've run the tests again using the latest version of the Dart SDK from Flutter, which cover reducers and combining reducers, and the tests are currently passing.
@envious Are you also using combineTypedReducer? Could you please provide the signature of your Reducer?
If either of you are providing your own Reducer to the store, it must match the signature (State, dynamic) -> State. You cannot provide your own type signature for the action parameter.

My Store looks like this
 final store = new Store<AppState>(
  appStateReducer,
  initialState: new AppState.init(),
  middleware: createStoreMiddleware()
  );

class AppState {
  final FirebaseUser firebaseUser;
  final OnBoardingState onBoardingData;
  ....
}

AppState appStateReducer(AppState state, action) {
  return new AppState(
      firebaseUser: authReducer(state.firebaseUser, action),
      onBoardingData: onBoardingReducer(state.onBoardingData, action),
  );
}

Hrm, that's an interesting one! If you add the typing to the action in your appStateReducer then all should be good for Dart 2. E.g.
AppState appStateReducer(AppState state, dynamic action) {

Unfortunalty no -_-
Even if I add dynamic to AppState appStateReducer(AppState state, dynamic action) I still get

error is happend type '(OnBoardingState, LicensesLoadedAction) => OnBoardingState' is not a subtype of type '(OnBoardingState, dynamic) => OnBoardingState'

Thanks for the report! Sorry about these issues. I've found that all tests pass using the standalone Dart 2 SDK, but using Flutter test. Redux runs all its tests using the standalone SDK since it's a pure dart lib, but this might be hiding an error, or Flutter might be running into an error.
I've filed a bug on the repo: https://github.com/flutter/flutter/issues/15900

I encounter the same issue
I/flutter ( 5391): type '(IdentityState, LoginSuccessfulAction) => IdentityState' is not a subtype of type I/flutter ( 5391): '(IdentityState, dynamic) => IdentityState on
Flutter 0.2.5-pre.55 • channel master • https://github.com/flutter/flutter.git
Framework • revision 39eeec47d6 (13 hours ago) • 2018-03-27 15:27:24 -0700
Engine • revision 3e877d371a
Tools • Dart 2.0.0-dev.40.0.flutter-06949dc985

Thanks for the report @dodyg!
As a heads up, I'm waiting on a response from the Flutter team to see if they can help us out here. Hard to know if this is a legit type issue that needs fixing or a Dart 2.

Hey @DrivEddy, @envious & @dodyg  -- Just a heads up that I've released a new version of Redux which addresses this problem. It also simplified the Redux API a bit, but requires a (pretty easy) migration. Please see the README for details: https://github.com/johnpryan/redux.dart/#redux-300-migration--dart-2-support


Using antiforgery with AJAX requests

I'd like to request a DELETE using a button on a form wired up to ...
$.ajax({
    url: '/delete/' + resource,
    type: 'DELETE',
    async: true,
    cache: false,
    success: function(result) {
        ...
    },
    error: function(jqXHR, exception) {
        ...
    }
});

... with ...
[HttpDelete("/delete/{resource}")]
public async Task<HttpStatusCode> Delete(string resource)
{
    ...
    return HttpStatusCode.OK;
}

This works fine without ValidateAntiForgeryToken, but as soon as I go ...
[HttpDelete("/delete/{resource}"), ValidateAntiForgeryToken]

:boom: I get a 400 Bad Request.
Both the button and the script are within a <form> and I can see the antiforgery token going out on the DELETE in request headers (via Fiddler) ...
Cookie: X-XSRF-COOKIE=CfDJ8Ag6EezFPexKq5ZuFJ4kc............;

What am I missing?

See the sample here: https://github.com/aspnet/Antiforgery/tree/dev/samples/AntiforgerySample
Are you sending the data as JSON or as form-url-encoded?
Our antiforgery system uses two tokens, a cookie token and a request token. The request token needs to be provided, and needs to match the cookie token,. You can do this either with a form field or with a header value (like the sample).
The reason why the request token needs to be there is that cookies are automatic. Another malicious site could ask a user's browser to hit your site and the cookie will be sent as part of that request. However, the malicious site has no way to get at the request token without an XSS exploit.

Ah ... I see.
This is form-url-encoded with the resource to be deleted in the route (e.g., https://www.mysite.com/controller/delete/d3df34d323223). Since it works fine without the validation attribute, it must just be the missing request token.
Thanks :smile: ... I think I have enough to get it fixed. I'll re-open if things don't pan out.

In case anyone comes across this, the solution was to add the token to the request as @rynowak said above. I couldn't get it to work as a header, and the sample is difficult to grok b/c its Angular and middleware and not a regular MVC sample.
I was able to get it to work by sending it in the data, like this ...
$.ajax({
    url: '/delete/' + resource,
    type: 'DELETE',
    async: true,
    cache: false,
    data: '__RequestVerificationToken=' + $('input[name=__RequestVerificationToken]').val(),
    success: function(result) {
        // You'll get a result == 200 here if you return a 200 Ok from your Delete method
    },
    error: function(jqXHR, exception) {
        // Trap exception here
        // jqXHR.status will contain the statuscode
        // exception will contain the message
    }
});


Why does the debug log message read "Skipping cache write for URL" when the URL is being cached?

It could be just me but I don't find the message produced here very intuitive: https://github.com/reclosedev/requests-cache/blob/ae95d6d96340fde8d65ed0c5c820f0be87d25346/requests_cache/session.py#L182
Obviously maybe I am reading it wrong or perhaps I misunderstood some basic concept here.

And since logging is one of the only things not covered by unit tests, any other feedback you have on logging would be helpful, or any suggestions for things that aren't being logged but should be.

Ok, I've only just now started using logging with requests-cache but I'll
keep an eye out. Thanks for fixing this so quickly!
On Wed, 15 Sept 2021 at 17:38, Jordan Cook @.***> wrote:

And since logging is one of the only things not covered by unit tests, any
other feedback you have on logging would be helpful, or any suggestions for
things that aren't being logged but should be.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/reclosedev/requests-cache/issues/408#issuecomment-920132000,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AASYMN5YPSI4TWWJKZPNFFLUCC4XTANCNFSM5ECTT62A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


picker options not working

for example i have this mapping
map("n", "<leader>fi", function()
  require("telescope").extensions.tailiscope.all({ initial_mode = "normal" })
end)

i expect to be in normal mode when i open the picker.
also i tried it in the setup and it doesn't work either
tailiscope = {
  initial_mode = "normal",
}

Should be fixed! Thanks for bringing this to my attention. Let me know if it works for you

with the mapping example it works, but in the setup it doesn't, also i have this mapping:
map("n", "<leader>fi", function()
  require("telescope").extensions.tailiscope.all(require("telescope.themes").get_dropdown({
    initial_mode = "normal",
    layout_strategy = "vertical",
    layout_config = { height = 0.95 },
  }))
end)

initial_mode works but the theme didn't change
if i were to do it in the setup it will look like this
tailiscope = {
  initial_mode = "normal",
  theme = "dropdown",
  layout_strategy = "vertical",
  layout_config = { height = 0.95 },
}

but the setup options don't work

Fixed the setup issue! Just passed values in the wrong order. Not sure why themes aren't working, but I'll definitely look into it.

works now thanks! checkout https://github.com/smartpde/telescope-recent-files implementation, i remember they had a similar issue and they fixed it

works now thanks! checkout https://github.com/smartpde/telescope-recent-files implementation, i remember they had a similar issue and they fixed it

Thanks for the resource! So it looks like after my last change themes work but break preview. It's probably got something to do with preview cutoff. I'll investigate in that repo to see if they have any solutions.


CppMicroServices#342 fix issue with missing paths when CppMicroServic…

On issue #342
Fix CMakeLists.txt files

Codecov Report

Merging #347 into development will increase coverage by 0.05%.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           development     #347      +/-   ##
===============================================
+ Coverage        89.28%   89.33%   +0.05%
===============================================
  Files               89       89
  Lines             6225     6266      +41
===============================================
+ Hits              5558     5598      +40
- Misses             667      668       +1

Impacted Files
Coverage Δ

...mework/include/cppmicroservices/ServiceInterface.h
95.78% <0%> (-0.51%)
:arrow_down:

framework/src/util/FrameworkPrivate.cpp
77.91% <0%> (+0.61%)
:arrow_up:

Hi, thanks for working on this. I started to review the changes but need some more time. I will get back to you soo.

I had a closer look and pushed another branch which should fix this issue in a more robust way, see https://github.com/CppMicroServices/CppMicroServices/compare/342-subproject-support for the changes.
If all checks run successfully, I will merge this today. Thanks for bringing this up.

This has been fixed in a57de72.


Updated dependencies to pick up latest manageiq-appliance_console

Updated dependencies to pick up latest (upcoming) manageiq-appliance_console 5.1.0 which would include the options for configuring the appliance for SAML Authentication.
https://bugzilla.redhat.com/show_bug.cgi?id=1767108

Ivanchuk backport details:
$ git log -1
commit 7dec167935821fbd528f2dd6e6b6e5e357c0d5bb
Author: Jason Frey<EMAIL_ADDRESS>Date:   Fri Nov 22 09:44:50 2019 -0500

    Merge pull request #264 from abellotti/updated_dependencies_for_appliance_console

    Updated dependencies to pick up latest manageiq-appliance_console

    (cherry picked from commit 435ef0792ab8fb6afb18155713ab16b043bbcf0c)

    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1775684


Too many logouts

Hello!
I'm tired enter your login and password every 20-30 minutes.

This should not happen, I never need to re-login. What are your cookie settings in this window?

@tjanson WOW! I didn't know about it section. Thank you, I'll try it :)


Allow user-defined entry.Time

Hi,
I'm currently writing a wrapper to go around logrus loggers.  This wrapper stores all logs in memory until it is explicitly flushed, at which point it logs to the underlying logger.  The intended use case is in tests, where I'm only interested in the logs for tests that fail.
The problem I'm having is that I would like the timestamps in the logs to reflect the time at which my program logs to the wrapper, not the time at which the wrapper is flushed.  I thought I could achieve this by modifying the Time field of an entry object.  However, it turns out that entry.Time is always set to time.Now() whenever one of the logging methods is called (see here).
I was hoping that we could avoid overriding entry.Time in entry.log() if it's nonzero.

For your use case, you can use a dedicated field for your initial timestamp.

That is a possible solution, however it also requires us to have special handling for the custom field, either in our formatters, or in our log consumers.  I hope you can see why this is sub-optimal for us.

Fixed by #781


⚠ PrivateHD has degraded performance

In e008e01, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 2099 ms

Resolved: PrivateHD performance has improved in c73f62c.


Bounty for CozaStore version of _CookieConsentPartial.cshtml

Ciao ragazzi,
I would need somebody to develop a nice looking CozaStore version of _CookieConsentPartial.cshtml, which

is responsive (i.e. works with Desktop & Mobile layout)
lets the user still choose the language

In addition to that I would need a minimal css to be able to overwrite the colours, because my shop is black/burgundy/white and so I have to substitute the blue CozaStore colours by something, which fits with my design.
Anyone interested?
Best regards,
Florian

In desktop I had noticed issues with it disappearing. I gave mine a high Z-index as the menu was always in higher.
<nav id="cookieConsent" class="navbar fixed-top navbar-light bg-light" role="alert" style="z-index:504440">


Plugins: Callbacks API

Implements Callbacks API.
Added permssions: Callbacks, CallbacksCustom, CallbacksDirect
Changes are mostly in plugins.src, few required changes were made in marinette.src, marilib.src

Closing this as plugin system is being rewritten due to severe security issues


Empty string in StringPreference serializes to 'None'

As in the title. I think it's because for some reason when the string preference is set to an empty string, it comes back from the db as None, which is then serialized as a string with return str(value).
This basically makes it impossible to set an empty string in a StringPreference.

Thanks for the report, you indeed found a bug.
You can solve this in your own project by doing this:
from dynamic_preferences.serializers import StringSerializer

class FixedStringSerializer(StringSerializer):
    @classmethod
    def to_python(cls, value, **kwargs):
        # this is the important bit that is currently missing
        if not value:
            return ''
        try:
            return str(value)
        except: pass
        try:
            return value.encode('utf-8')
        except: pass
        raise cls.exception("Cannot deserialize value {0} tostring".format(value))

class YourPreference(StringPreference):
    serializer = FixedStringSerializer

I'll try to publish a release including a patch for this during the weekend.
Sorry for the trouble!

No trouble, I was about to work on a PR if you haven't already started on it?

Sure a PR is welcome (include a unittest if you can, otherwise  I'll do it myself later)!

Ok, right on it - I'll add a test as well.


Clarify wording of crypto_keylength. Fixes #421

Signed-off-by: David A. Wheeler<EMAIL_ADDRESS>

Lgtm


Usage with React

I've been using render-media to render streams (both webtorrent, and images) into our code and it works great. I now have to embed something into a React application.
Unfortunately react frowns on pretty much anything else touching the HTML tree, so i'm trying to figure out how to use Render-Media with a react component - note that I don't need to actually build a react-component for render-media, its more of a question of whether anyone has figured out how to use render-media within a react-component.

OK - complicated ... but got it to work like
import React from 'react'
import PropTypes from 'prop-types'
export default class extends React.Component {
    static propTypes = { ...};
    constructor(props)
    {
        super(props);
        this.state = {  ... }
        this.loadImg = (enclosingspan) => // Defined as a closure so that can access identifier
            const file = {
                name: name,
// Your custom code that returns something like fs.createReadStream(opts)
                createReadStream: f_createReadStream(this.identifier)
        };
        RenderMedia.append(file, enclosingspan, cb);  // Render into supplied element - have to use append, as render doesnt work on firefox for images, the cb will set attributes and/or add children.
    }

    render() {
            return
    }
}

Just saw this issue. Glad you got it figured out @mitra42!


fix code formatting in documentation

Hello, thanks for this library!
Here's a quick MR to fix a code block appearing as text in the documentation.
Thanks

Thank you!


document that we delete merged branches

We enable Automatically delete head branches since a long time so we
should also document it.

Travis seems stuck 😭


[BUG] UniGetUI asks for parse_pip_search on every launch

Please confirm these before moving forward

[X] I have searched for my issue and have not found a work-in-progress/duplicate/resolved issue.
[X] I have tested that this issue has not been fixed in the latest (beta or stable) release.
[X] I have checked the FAQ section for solutions.
[X] This issue is about a bug (if it is not, please use the correct template).

UniGetUI Version
3.1.5
Windows version, edition, and architecture
Windows 11
Describe your issue
After upgrading to UniGetUI 3.1.5, I get a message about installing
python -m pip install parse_pip_search
If I choose to install it, then after I restart UniGetUI it still says it is missing and prompts me to install it again.
I tried also manually installing it in command prompt and also used --force-reinstall option, but UniGetUI still does not like it.
Steps to reproduce the issue
No response
UniGetUI Log
[22.12.2024 17:25:59]    __  __      _ ______     __  __  ______
                        / / / /___  (_) ____/__  / /_/ / / /  _/
                       / / / / __ \/ / / __/ _ \/ __/ / / // /
                      / /_/ / / / / / /_/ /  __/ /_/ /_/ // /
                      \____/_/ /_/_/\____/\___/\__/\____/___/
                          Welcome to UniGetUI Version 3.1.5
[22.12.2024 17:25:59]
[22.12.2024 17:25:59] Build 74
[22.12.2024 17:25:59] Encoding Code Page set to 866
[22.12.2024 17:25:59] Using built-in UniGetUI Elevator
[22.12.2024 17:26:00] Loaded language locale: ro
[22.12.2024 17:26:00] Randomly-generated background API auth token for the current session: 2ced2ny5on7che2ryjadsg2ednwd0m502t0rxkh9ze781zw7htgdk47jb5i1anoe
[22.12.2024 17:26:00] Api running on http://localhost:7058
[22.12.2024 17:26:00] Lang files were updated successfully from GitHub
[22.12.2024 17:26:01] Command npm was not found on the system
[22.12.2024 17:26:01] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                      █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                      █ Name: Npm
                      █ Enabled: True
                      █ Found: False
                      █ THE MANAGER WAS NOT FOUND. PERHAPS IT IS NOT
                      █ INSTALLED OR IT HAS BEEN MISCONFIGURED
                      ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[22.12.2024 17:26:01] Command vcpkg was not found on the system
[22.12.2024 17:26:01] Command cargo was not found on the system
[22.12.2024 17:26:01] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                      █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                      █ Name: Cargo
                      █ Enabled: True
                      █ Found: False
                      █ THE MANAGER WAS NOT FOUND. PERHAPS IT IS NOT
                      █ INSTALLED OR IT HAS BEEN MISCONFIGURED
                      ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[22.12.2024 17:26:01] Command vcpkg was not found on the system
[22.12.2024 17:26:01] Command vcpkg was not found on the system
[22.12.2024 17:26:01] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                      █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                      █ Name: vcpkg
                      █ Enabled: True
                      █ Found: False
                      █ THE MANAGER WAS NOT FOUND. PERHAPS IT IS NOT
                      █ INSTALLED OR IT HAS BEEN MISCONFIGURED
                      ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[22.12.2024 17:26:01] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                      █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                      █ Name: Pip
                      █ Enabled: True
                      █ Found: True
                      █ Fancy exe name: pip
                      █ Executable path: C:\Program Files\Python312\python.exe
                      █ Call arguments:  -m pip
                      █ Version:
                      █   pip 24.3.1 from C:\Users\Mihai\AppData\Roaming\Python\Python312\site-packages\pip (python 3.12)
                      ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[22.12.2024 17:26:01] Downloaded new icons and screenshots successfully!
[22.12.2024 17:26:02] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                      █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                      █ Name: Winget
                      █ Enabled: True
                      █ Found: True
                      █ Fancy exe name: winget.exe
                      █ Executable path: C:\Users\Mihai\AppData\Local\Microsoft\WindowsApps\winget.exe
                      █ Call arguments:
                      █ Version:
                      █   System WinGet CLI Version: v1.9.25200
                      █   Using Native WinGet helper (COM Api)
                      ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[22.12.2024 17:26:02] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                      █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                      █ Name: .NET Tool
                      █ Enabled: True
                      █ Found: True
                      █ Fancy exe name: dotnet tool
                      █ Executable path: C:\Program Files\dotnet\dotnet.exe
                      █ Call arguments: tool
                      █ Version:
                      █   9.0.101
                      ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[22.12.2024 17:26:03] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                      █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                      █ Name: Chocolatey
                      █ Enabled: True
                      █ Found: True
                      █ Fancy exe name: choco.exe
                      █ Executable path: C:\Users\Mihai\AppData\Local\UniGetUI\Chocolatey\choco.exe
                      █ Call arguments:
                      █ Version:
                      █   2.2.2
                      ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[22.12.2024 17:26:03] Command scoop was not found on the system
[22.12.2024 17:26:03] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                      █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                      █ Name: Scoop
                      █ Enabled: True
                      █ Found: False
                      █ THE MANAGER WAS NOT FOUND. PERHAPS IT IS NOT
                      █ INSTALLED OR IT HAS BEEN MISCONFIGURED
                      ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[22.12.2024 17:26:04] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                      █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                      █ Name: PowerShell
                      █ Enabled: True
                      █ Found: True
                      █ Fancy exe name: powershell.exe
                      █ Executable path: C:\WINDOWS\system32\windowspowershell\v1.0\powershell.exe
                      █ Call arguments:  -NoProfile -Command
                      █ Version:
                      █   Name                           Value
                      █   ----                           -----
                      █   PSVersion                      5.1.26100.2161
                      █   PSEdition                      Desktop
                      █   PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
                      █   BuildVersion                   10.0.26100.2161
                      █   CLRVersion                     4.0.30319.42000
                      █   WSManStackVersion              3.0
                      █   PSRemotingProtocolVersion      2.3
                      █   SerializationVersion           <IP_ADDRESS>
                      ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[22.12.2024 17:26:05] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                      █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                      █ Name: PowerShell7
                      █ Enabled: True
                      █ Found: True
                      █ Fancy exe name: pwsh.exe
                      █ Executable path: C:\Users\Mihai\AppData\Local\Microsoft\WindowsApps\pwsh.exe
                      █ Call arguments:  -NoProfile -Command
                      █ Version:
                      █   PowerShell 7.4.6
                      ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[22.12.2024 17:26:05] LoadComponentsAsync finished executing. All managers loaded. Proceeding to interface.
[22.12.2024 17:26:06] Found 52 installed packages from Pip
[22.12.2024 17:26:06] Found 1 installed packages from .NET Tool
[22.12.2024 17:26:06] Found 0 installed packages from PowerShell7
[22.12.2024 17:26:06] Command parse_pip_search.exe was not found on the system
[22.12.2024 17:26:06] Dependency parse-pip-search was not found for manager Pip, marking to prompt...
[22.12.2024 17:26:07] Dependency .NET Tools Outdated for manager .NET Tool is present
[22.12.2024 17:26:07] Found 0 installed packages from Chocolatey
[22.12.2024 17:26:07] Found 0 available updates from Chocolatey
[22.12.2024 17:26:09] Found 1 installed packages from PowerShell
[22.12.2024 17:26:09] Found 0 available updates from PowerShell7
[22.12.2024 17:26:09] Found 0 available updates from .NET Tool
[22.12.2024 17:26:11] Found 1 available updates from PowerShell
[22.12.2024 17:26:12] Found 4 available updates from Winget
[22.12.2024 17:26:12] Found 367 installed packages from Winget
[22.12.2024 17:26:12] Unknown mimetype text/html; charset=utf-8 for icon https://github.com/darktable-org/darktable/blob/master/data/pixmaps/256x256/darktable.png, aborting download
[22.12.2024 17:26:13] Unknown mimetype text/html; charset=UTF-8 for icon https://postimg.cc/T5KzZ7NR, aborting download
[22.12.2024 17:26:13] Icon download attempt at https://community.chocolatey.org/content/packageimages/MobaXTerm.23.0.0.png failed with code NotFound
[22.12.2024 17:26:14] Could not parse Images array from Microsoft Store response
[22.12.2024 17:26:14] Icon download attempt at https://community.chocolatey.org/content/packageimages/openhashtab.3.0.2.png failed with code NotFound
[22.12.2024 17:26:14] Icon download attempt at https://community.chocolatey.org/content/packageimages/opentrack.2022.3.2.png failed with code NotFound
[22.12.2024 17:26:14] Icon download attempt at https://community.chocolatey.org/content/packageimages/sunshine.0.18.3.png failed with code NotFound
[22.12.2024 17:26:14] Found 0 available updates from Pip
[22.12.2024 17:26:14] Icon download attempt at https://upload.wikimedia.org/wikipedia/en/thumb/7/7e/Wacom_Logo_WhiteType.svg/254px-Wacom_Logo_WhiteType.svg.png failed with code NotFound

Package Managers Logs
Manager WinGet with version:
System WinGet CLI Version: v1.9.25200
Using Native WinGet helper (COM Api)

——————————————————————————————————————————

Logged native task on manager Winget. Task type is ListSources
Process start time: 22.12.2024 17:26:01
Process end time:   22.12.2024 17:26:02

-- Task information
 ...

The task reported success

——————————————————————————————————————————

Logged native task on manager Winget. Task type is ListInstalledPackages
Process start time: 22.12.2024 17:26:05
Process end time:   22.12.2024 17:26:12

-- Task information
 ...

The task reported success

——————————————————————————————————————————

Logged subprocess-based task on manager Winget. Task type is RefreshIndexes
Subprocess executable: "C:\Users\Mihai\AppData\Local\Microsoft\WindowsApps\winget.exe"
Command-line arguments: " source update --disable-interactivity"
Process start time: 22.12.2024 17:26:06
Process end time:   22.12.2024 17:26:07

-- Process STDOUT
 ...

Return code: SUCCESS (0)

——————————————————————————————————————————

Logged native task on manager Winget. Task type is OtherTask
Process start time: 22.12.2024 17:26:06
Process end time:   22.12.2024 17:26:10

-- Task information
 ...

The task reported success

——————————————————————————————————————————

Logged native task on manager Winget. Task type is ListUpdates
Process start time: 22.12.2024 17:26:07
Process end time:   22.12.2024 17:26:12

-- Task information
 ...

The task reported success

——————————————————————————————————————————

Relevant information
No response
Screenshots and videos
No response

Duplicate of https://github.com/marticliment/UniGetUI/issues/3079.
You need to add the Python's "Scripts" folder to path

Thanks! I haven't seen #3079 because of the underscores vs dashes in parse_pip_search when searching.
I added C:\Users\Mihai\AppData\Roaming\Python\Python312\Scripts to my local user PATH variable and now it works. Maybe consider adding a hint in the error message?

Maybe consider adding a hint in the error message?

That is a good idea, I'll see what I can do


Run migration as DB transactions

Summary / User Story
Currently Orchestrate migrations are executed sequencially in a non-transactional way therefore in case of failure migrations are partially applied creating a potential inconsiste DB state.
Technical Notes

https://github.com/go-pg/migrations/blob/v8/README.md#transactions

Business Value
Prevent client to run in inconsisten DB states which will derive in a high support cost to sort that such of cases.
Acceptance Criteria

[ ] In case a DB migration fails, DB state remains as before migration was executed

Migrations already run in DB txs


Add setting to configure ecdh curve

fixes #417

This change is

Coverage decreased (-0.02%) to 43.83% when pulling 8e41bdd3d46adaa54d28da50bd2e06463abadf5c on aledbf:ssl_ecdh_curve into c5949252ede97a8fb78464c6e801518a51309491 on kubernetes:master.


Docs updates for Trusty rebrand

Summary
Docs updates for the rebranding of Trusty to Stacklok Insight. Also adds @docusaurus/plugin-client-redirects for redirect support.
Change Type
Mark the type of change your PR introduces:

[ ] Bug fix (resolves an issue without affecting existing features)
[ ] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[x] Documentation (updates or additions to documentation)
[ ] Refactoring or test improvements (no bug fixes or new functionality)

Testing
Outline how the changes were tested, including steps to reproduce and any relevant configurations.
Attach screenshots if helpful.
Review Checklist:

[x] Reviewed my own code for quality and clarity.
[x] Added comments to complex or tricky code sections.
[x] Updated any affected documentation.
[ ] (N/A) Included tests that validate the fix or feature.
[x] Checked that related changes are merged.

coverage: 55.205% (-0.01%) from 55.215%
when pulling 0f471ddb6523fd37f9aaff433e7cce3617f7c080 on danbarr:trusty-rebrand
into b4fc9e370f8d3996a4ed8fc417aeae3a8eb84bcd on mindersec:main.


Feature: Home Page Logo

Hi Anton, I was trying to put an image between the side menu button and the site's header.

Is there a way to achieve that?
Thanks

Hi, you can try to use mark_safe
Like
from django.utils.safestring import mark_safe
MATERIAL_ADMIN_SITE = {
'HEADER':  mark_safe(' Donde comemos | Panel de ...'),
...
}

Thank you Anton, i would like to make this as a setting later so when i have some time i'll make a PR with this feature.
Its a good solution, for now.


Upgrade modeljson ints to int64 if their proto uses int64

This is part of https://github.com/elastic/apm-data/issues/118.
Some proto definitions are using int64 when their modeljson has been using int.
I'm doing it this way since the flow is modelpb->modeljson.

I'm opening the PR this way after discussing it quickly with @kruskall due to the way the transformation happens.
However, for all these attributes, it seems unlikely the count needs to be an int64. So I'd be fine using int32 in the protobuf definitions too.

However, for all these attributes, it seems unlikely the count needs to be an int64. So I'd be fine using int32 in the protobuf definitions too.

Maybe should, not sure - but I think keeping them aligned makes sense either way, so we could do that as a separate change.


Support pytest 7

When installing pytest-aiomoto: pytest-aiomoto 0.4.0 depends on pytest<7.0.0 and >=6.2.5

Thanks for submitting this request.  To date, this project is only tested with pytest 6.x.  The pytest 7.x release is a recent major version release that could break things in this current project.  If I don't look into this soon, please submit a PR on this repo with an update to pytest 7.x in the pyproject.toml specs, and other updates for any pytest plugins that also work with 7.x.
To further evaluate an update to pytest 7.x, I would also need to use a pre-release of this package in the aio-aws project.  Doing that would provide some data points about the downstream impact of updating pytest to 7.x.

$ make package-publish
All set!
Building pytest-aiomoto (0.5.0)
  - Building sdist
  - Built pytest-aiomoto-0.5.0.tar.gz
  - Building wheel
  - Built pytest_aiomoto-0.5.0-py3-none-any.whl
Checking dist/pytest_aiomoto-0.5.0-py3-none-any.whl: PASSED
Checking dist/pytest-aiomoto-0.5.0.tar.gz: PASSED

Publishing pytest-aiomoto (0.5.0) to PyPI
 - Uploading pytest-aiomoto-0.5.0.tar.gz 100%
 - Uploading pytest_aiomoto-0.5.0-py3-none-any.whl 100%

Examples of upgrading to 0.5.x can be found in https://github.com/dazza-codes/aio-aws/pull/89

e.g. the aio*server fixtures return the service object, not use the endpoint URL


TypeError: init() got an unexpected keyword argument 'connect_timeout'

Hello, running ubuntu 18. Using Python 3.6.9
This looking to be related to http2.0 anyone else having a problem with this?
$ python server.py
Traceback (most recent call last):
  File "server.py", line 450, in <module>
    timeout = httpx.Timeout(15.0, connect_timeout=30.0)
TypeError: __init__() got an unexpected keyword argument 'connect_timeout'

Just attempted to replicate your issue. Received the following error while attempting to install httpx module into a python 3.6.9 environment: The currently activated Python version 3.6.9 is not supported by the project (^3.7). Trying to find and use a compatible version. So my guess is your python version is below the current requirements of python 3.7. The official doc does say python 3.6+, but may need to be updated. Otherwise you will need to edit the timeout reference.

Attempting to replicate. In the meantime, do me a favour and install / update your python to at least 3.8 and see if the issue persists.

Issue corrected. httpx revision changed connect timeout argument from connect_timeout to just connect. Should be good to go now. download the new code and let me know if it works correctly.

Solved! Thanks @rybodiddly


BUG: Cropped Certificate Thubnails

Describe the bug

On medium screen in portrait, the certificate thumbnails are cut off.
Reproducing the bug**

Open site
Navigate to credentials page
Rotate screen to portrait

Expected behavior

The thumbnails should not be cropped.
Screenshots

Platforms

Device: iPad Pro
OS: iPadOS
Browser: Firefox
**Version:**All

Additional context

Use lg instead of md and then add new sizing for md

#242


get_password data is not support?

get_password data is not support?

You are right. Pull request is welcome.


tf: disable wait on YAML

If we do this long term we should probably remove the option entirely.
For now, just hack it out to see the impact.
Please provide a description of this PR:

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
0a43c3907d427b94bb1b078374cfdf9932d6fcd7
link
true
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
0a43c3907d427b94bb1b078374cfdf9932d6fcd7
link
true
/test lint_istio

gencheck_istio
0a43c3907d427b94bb1b078374cfdf9932d6fcd7
link
true
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Documentation suggestions

Hello.
Here I'm willing to suggest documentation improvements. I can help with it if you want. Here's some of them:

My system lacks /etc/sysctl.d/10-ptrace.conf: What should I put it instead?

I'm sometimes not sure how to use Reptyr with multiprocess programs

My system lacks /etc/sysctl.d/10-ptrace.conf: What should I put it instead?

Adding kernel.yama.ptrace_scope = 0 to any .conf file in /etc/sysctl.d (or a new one) will work, if your system has YAMA and not that config file. Do you have a /proc/sys/kernel/yama/ptrace_scope?

I'm sometimes not sure how to use Reptyr with multiprocess programs

reptyr -T is the only mode that will (may) work with multi-process programs, for somewhat complicated reasons.

Adding kernel.yama.ptrace_scope = 0 to any .conf file in /etc/sysctl.d (or a new one) will work, if your system has YAMA and not that config file.

Done. Thanks!
$ cat /etc/sysctl.d/10-ptrace.conf
# Disable it to make reptyr happy
 kernel.yama.ptrace_scope = 0

Do you have a /proc/sys/kernel/yama/ptrace_scope?

Yes
# cat /proc/sys/kernel/yama/ptrace_scope
1
# echo 0 > /proc/sys/kernel/yama/ptrace_scope
# cat /proc/sys/kernel/yama/ptrace_scope
0

reptyr -T is the only mode that will (may) work with multi-process programs, for somewhat complicated reasons.

Excuse me for my ignorance, but I can't make it work.

The process runs at /dev/tty3 (TTY number 3, is that /dev/tty3?)
$ reptyr -T 20039
[-] Child is not connected to a pseudo-TTY. Unable to steal TTY.
Unable to attach to pid 20039: Invalid argument

$ reptyr -T 20016
[-] Child is not connected to a pseudo-TTY. Unable to steal TTY.
Unable to attach to pid 20016: Invalid argument

$ reptyr -T 20015
[-] Child is not connected to a pseudo-TTY. Unable to steal TTY.
Unable to attach to pid 20015: Invalid argument

$ reptyr -s 20039
It works, but the terminal shows something "broken"

If I try to use it, more trash is shown. It's like keyboard input isn't used correctly..

/upgrade on WeeChat did the trick, but I had to write too many trash and do random tries until I was able to type it :P
My terminal emulator: Terminology


Nightscout does not recognise Bolus Wizard carbs from Medtronic 512 pump

Not sure if I'm posting in right place but....
Initially posted this on Facebook through Looped!
At @danamlewis suggestion posted this in Gitter openaps/hardware dev
Someone was going to look into it but this was a couple of weeks ago and issue not resolved.
Since then I have been calculating bolus using bolus wizard, and then doing a carb correction through Nightscout.
Would prefer not to have to do this if possible.
Is anyone able to look into this further please?
(using Medtronic 512 with Edison board rig)
Thanks in advance

This never worked for me with my 512 on Edison. So I usually just quickbolus, stay around 1/3 below my calculation and let oref0 basals do the rest. This is with Fiasp dosing before eating and wide basal ranges.
Of course, it's just a workaround, but relying on autosens works nicely for my rapidly changing sensitivity.

This should be fixed when one does a new setup with a 0.7.0 branch for the x12. Can someone with an x12 please confirm?

@cluckj: This totally worked! Thank you !!!
I was on dev, when switching to 0.7.0-dev. So this is not a newly flashed setup. But the switch worked without flashing vom scratch, which it didn't some weeks before. Also, as carbs never have worked before, this is not an 0.6 inherited fix.

Maybe I was too quick. When using bolus wizard the 2nd time, the bolus vanished completely. It's still seen on the pump display, but it's not part of loop-calculation or nightscout.
First working bolus was 0.5, 2nd failing was 3.0U. Could the problem be time span related?

Can you run rm ~/myopenaps/monitor/pumphistory-24h-zoned.json to force a full pump history refresh, and see if that makes the bolus show up in the loop calculation?

I did that. But even after a refresh, there was no change.
Then I found sth else, that might be relevant. In nightscout admin the first bolus expert was logged as future object in mdb and any basal change since then. The second bolus expert was just logged with the carbs (in future objects...). I delete them now.

Please be very careful to note whether you’re seeing problems with refreshing pump history records on the rig, and having them show up in IOB etc., or if you’re seeing problems with those treatments being uploaded to NS. Those are very different things, and only the former matters for looping safely.

I mentioned, because I was pretty sure about this specific incidence.
Rig calculation and NS data were in sync. 2h zero temp, bolus wizard 30g/3U, but 20 min. with 10U basal later around 2.9 IOB on rig as on NS.

P.S.: after deleting future treatments, all basals, boluses and carbs are gone since the first bolus expert some hours before (Nightscout), but IOB looks unchanged (OK).

is this still an issue with x12 pumps and 0.7.0-dev? Can someone clarify?

This was resolved in https://github.com/ecc1/medtronic/commit/3cc940777ec9bfea456fe2f0432dbf9ba97fbe25
Should be okay to close this issue.


aws: fix assertion failure in debug due to timer access outside of main thread

Commit Message: aws: fix assertion failure in debug due to timer access outside of main thread
Additional Description:
Patch to ensure async credential providers do not access main thread timers when credentials are requested. This was a larger fix than originally expected, due to substantial test case rewrites, changes to refresh logic and changes to the cluster initialisation sequence.
This patch performs the following:

Changes async providers to kick off credential refresh process via onClusterAddOrUpdate, replacing the init handler
Changes async providers to be truly async. They now perform credential refresh based on a timer, set to the expiration time returned in the credential payload, or to the default cache duration of 1 hour. Async providers will no longer trigger credential refresh via the data plane (which was actually the cause of the original bug). Async providers will start refreshing with a 2 second timer and doubling to 32 seconds (or until they succeed) avoiding load on STS or IMDS.
Fixes the cache duration calculation to be actually 1 hour, rather than 1 hour * 60 * 60 :)
Fixes a reported bug that also caused assertion failure in route specific configuration

Risk Level: Low
Testing: Unit
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue] https://github.com/envoyproxy/envoy/issues/33962
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

@suniltheta

A note on this PR - the majority of the change is to implement a handler for onClusterAddOrDelete, and that handler literally calls refresh() once the cluster is ready to handle requests. If there is a shorter and/or cleaner way to do this, please let me know. I've used code from dynamic forward proxy to do most of the onClusterAddOrDelete handling.

@suniltheta for first pass, thanks.
/wait

/retest

@suniltheta some minor refactoring - there was an initialization issue I found with upstream filter that I've now addressed.


Add a service_blacklist configuration to ensure certain services never have metrics collected

…r have metrics collected.
Closes #1186

What does this PR do?
Addsa service_blacklist configuration to ensure certain services never have metrics collected
Motivation
I was on a role making consul changes and saw https://github.com/DataDog/integrations-core/issues/1186 so I tackled it
Versioning

[x] Updated CHANGELOG.md. Please use Unreleased as the date in the title

@TylerLubeck Hello there! Would you mind rebasing and using the new filter utility that was introduced in https://github.com/DataDog/integrations-core/pull/1510?
I really appreciate it :bow:

Copying out of a closed comment thread:
I can use the pattern_filter function, but I think either there's a bug in that function or it doesn't do quite what we would need here:
https://github.com/DataDog/integrations-core/blob/master/datadog_checks_base/datadog_checks/utils/common.py#L47
I think that in the case that there's a blacklist, then the items have the blacklisted items removed but they're not restricted to just the whitelisted items.
I've pushed up the code change to start using the function anyway, if only for the sake of not neglecting this for a week

@ofek can you chime in?

@TylerLubeck You can chain the function one type of list at a time.
That is not a bug btw, but rather some tricky membership logic: the only way to use a WL/BL at the same time without either being completely ignored is by applying the BL without members of the WL.

Resolved by https://github.com/DataDog/integrations-core/pull/2174


Highlight incorrect syntax

As a feature request, it would be nice if I knew that my ansible was wrong before running it. Unquoted variables are the first culprit that comes to mind.

I think best would be use linter/ansible-playbook --syntax-check linter plugin for this


x/pkgsite: package removal request for github.com/erdaltsksn/go-response

What is the path of the package that you would like to have removed?
github.com/erdaltsksn/go-response
Are you the owner of this package?
Yes
What is the reason that you could not retract this package instead?
I removed the repository and moved the package inside the another go project's pkg folder. There aren't any dependents for this package.

Done.


[stable/consul] change configuration (enable ACLs)?

Is this a request for help?: Yes/Maybe

Is this a BUG REPORT or FEATURE REQUEST? (choose one): Feature request
I would like to be able to enable and configure ACLs on the cluster. I found no indication of how to achieve that. The default behaviour of Consul is to run without ACLs. I'd like for a master token to be generated and potentially stored in a secret in kubernetes when the cluster bootstraps if ACLs are enabled for the cluster.
Version of Helm and Kubernetes:
Helm version:
Client: &version.Version{SemVer:"v2.4.2", GitCommit:"82d8e9498d96535cc6787a6a9194a76161d29b4c", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.4.2", GitCommit:"82d8e9498d96535cc6787a6a9194a76161d29b4c", GitTreeState:"clean"}

Kubernetes version:
Client Version: version.Info{Major:"1", Minor:"6", GitVersion:"v1.6.4", GitCommit:"d6f433224538d4f9ca2f7ae19b252e6fcb66a3ae", GitTreeState:"clean", BuildDate:"2017-05-19T20:41:07Z", GoVersion:"go1.8.1", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"6", GitVersion:"v1.6.2", GitCommit:"477efc3cbe6a7effca06bd1452fa356e2201e1ee", GitTreeState:"clean", BuildDate:"2017-04-19T20:22:08Z", GoVersion:"go1.7.5", Compiler:"gc", Platform:"linux/amd64"}

Which chart:
stable/consul
What happened:
ACLs can't be enabled.
What you expected to happen:
Configuration option to enable ACLs in the cluster.
How to reproduce it (as minimally and precisely as possible):
Deploy the chart
Open the UI
Try to access the ACL tab

@mitom I'd be happy to do this. Can you confirm the flags that need to enable ACLs?

@lachie83 The basic configuration is explained on consul's page
The very minimum to set it up is to provide acl_datacenter, there can only be one authoritative DC even when consul is set up cross-dc. I don't think this would cause any issues here, we can just set it to be dc0 (the default value for dc for consul clusters that don't specify any).
The slightly more annoying part is setting the master token (acl_master_token). This can be any value (consul uses UUIDs as tokens, the master doesn't have to be one, but it would be convenient for consistency), but should be reasonable complex (as in long) since it will essentially have admin privileges. All of the servers need to have access to it, and when it changes the configuration needs to be reloaded on all the servers and the current leader needs to restart (to force a new leader-election). I imagine we could just create a new deployment and throw away the old containers which would satisfy this.
This also needs to be available to the user at least once, to use the ACL part of the interface one needs to specify a token. Since at the beginning there is only this master token and an anonymous one with no privileges to access that part, this is required to create more tokens (same applies to managing tokens through any other interface, this master token is needed to bootstrap the cluster properly).
The thing is, there are a lot more configuration options (not just about ACLs but in general) - maybe it would be worth to solve this via allowing to load in extra configuration files in some way?
Consul supports providing a directory for configurations via -config-dir in which it will load all .json files in alphabetical order (later overrides earlier). The position of specifying -config-dir and -config-file matters as well, as in the case of
consul agent -config-dir /opt/consul/config -config-file /example.json

example.json will override the configs in /opt/consul/config
while in the case of
consul agent -config-file /example.json -config-dir /opt/consul/config

the opposite is true.
Generally speaking, this would be easily solvable by allowing a list of confimaps to be passed, mounting them in a directory and then loading that directory before loading the generated things, like the gossip key config. The shortcoming of this is that the acl_master_token is highly sensitive data, so just taking it from a configmap is not really a way to go. There are also some other tokens  similar to it related to acls (they are all optional to set up, but there are use cases):

acl_agent_master_token - used when the authoritive DC is not available, should not matter for us since there is only 1 DC. Should only be used during outages.
acl_agent_token - used for internal cluster operations, defaults to acl_token
acl_token - used for client requests
acl_replication_token - used for ACL replication to other DCs, should not matter for us as there is only 1 DC.

These all range in terms of sensitivity but some - by definition - require high privileges. They are also meant to be set up by the cluster administrator once ACLs have been bootstrapped, but their values would need to be passed to the servers in some form of configuration. And there are also non-sensitive information, like acl TTLs and such.
Essentially, there should be a way to provide both sensitive and non-sensitive configuration. My best idea is to provide the ability to

Set a list of configmaps in helm which will be mounted as files (say /opt/consul/config as an example)
Set a list of secrets in helm which will be mounted in the same folder as the configs.

In this scenario, someone could create configmaps named like 00_cluster_config (.json..? I'm not sure what's the better practice there) which could be mounted as /opt/consul/config/00_cluster_config.json that contain general config, like ACL ttls and such and then a secret named 99_cluster_secret_config (.json..? once again, not sure what's better in k8s naming terms). In this case the 2 configs would complement each other. It would also be possible to do things like specifying the different tokens in different secrets to control who can view what in k8s.
That's of course just an idea and would need to be tested, but that's what I could come up with to support all the configuration best and would stay relatively light in terms of the chart.
Let me know what you think!

@mitom this is great thanks! Obviously happy to review a PR if you manage to get to this before me. I don't have many free cycles currently.

@lachie83 submitted a PR, there is 1 annoying thing in it with --set, see the PR for a explanation. Let me know what you think!


Add argument to specify plugin path

We're generating Dart protobuf files and need to be able to explicitly specify the path to the protoc-gen-dart plugin. This adds an option to the Maven plugin to specify such a path.

Ack. Thanks.
Shouldn't this line:
cmd.add("--plugin=protoc-gen-" + type + "=" + pluginPath);
be simply:
cmd.add("--plugin=" + pluginPath);
?

It could be done like in your suggestion to make it more generic. For an output type plugin the protoc-gen-TYPE= part is actually necessary.

Okay, sounds good (I'm not so familiar with protoc plugins)


ORC-1350: Upgrade setup-java to v3

What changes were proposed in this pull request?
This PR aims to upgrade setup-java from v1 to v3. v3 has two breaking changes.

JDK distribution field is required
1.8 is not accepted. We need to use 8 instead.

Why are the changes needed?
Currently, we are using v1 which is too old and shows warnings like the following.

https://github.com/apache/orc/actions/runs/3871285820/jobs/6598946894

Warning: The `set-output` command is deprecated and will be disabled soon.
Please upgrade to using Environment Files. For more information see:
https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

The latest one is v3.9.0. We will use it.

https://github.com/actions/setup-java/releases/tag/v3.9.0

Note that Node.js is upgraded since v3.0.
https://github.com/actions/setup-java/releases/tag/v3.0.0

In scope of this release we changed version of the runtime Node.js for the setup-java action and updated package-lock.json file to v2.

How was this patch tested?
Manually check the GitHub Action logs.

Could you review this too, @guiyanakuang ?

Merged to main.

Thank you for review and merging, @guiyanakuang !

I backported this to branch-1.8 as a part of Apache ORC 1.8.2 release preparation.


matched icons and reworked entity generation

Seeing as you actively maintain warehousing again I wanted to share some changes I made for myself.
Storehouse icons are changed to a visual style similar to warehouses being less colorful and sporting a dot indicating logistic type.
Generating all entities by hand is prone for typos so I turned generation into a parametrized function duplicating logistic versions from the basic ware- and storehouses.

Well, @dgw is doing most of the maintaining these days, but it's still an active mod!
I'm looking at your generation functions and well, they seem pretty similar to what is already being done, except rolling up some things, and when adding being a bit more verbose.
I'll leave it to @dgw if he likes your changes, at the end of the day as long as the tables getting generated are the same then, eh, doesn't matter.
Icons look good, and is probably a good idea as well

I actually think @Nova-Kast is the best one to weigh in on the new icons, since he's working on graphics tweaks for Factorio 0.17. To my eye, they look too dark, but I actually don't know if the storehouses are part of Nova_Kast's plans. I'll have to ping him.
The code diff is pretty overwhelming to review on my phone, so I'll put that off for now. :grinning:

Merging such old pull requests always is a chore. I hope I got all of those tiny change requests.

I really appreciate the quick turnaround, @Yousei9!
Assuming I can get my Lua interpreter going, I'll try to verify that the old and new code produce equivalent output without actually loading the mod (because I'm much too lazy to install Factorio and start a test game just for this :grin:).
If all is well, would you prefer that I handle squashing the tweaks you added today into the appropriate commit, or do you want to do that yourself? I know this is an old PR, so I'm happy to do the rebase for you if you'd rather move on to other things.

I totally overlooked "-shadow" had been added to warehouse sprites. Feels very inconsistent to me. To reduce overall mod filesize you may want to break apart sprite and shadow like all base factorio entities do.
Removed the -shadow from the sprites so those names can safely be used for shadow sprite.
I very much prefer unscaled icons myself. I'll add a mod option to toggle it on/off defaulting to on.
I don't recall why I removed hardness and changed health, but neither of them really impact gameplay in a huge way. If you want I'll add health as parameter to the creation function.
scale_entity_info_icon is a property of crafting machine not chests
https://wiki.factorio.com/Prototype/CraftingMachine

The -shadow suffix on sprite filenames was left over from… I have no idea when. It was there when I got involved with the mod about a year ago, and had probably been there for ages already. Neither I nor @Anoyomouse can regenerate the sprites from the source models (we don't have them), so any thoughts of splitting the graphics into separate files will have to wait for @Nova-Kast to come out of hibernation.
I do like that this rewrite got rid of a bunch of dead code (the circuit-connector functions) that was no longer used. That's a big plus.
The new setting… Honestly, adding a setting is just out of scope for this PR, which is to rewrite the entity-generation code and change a few icons. The icon updates, even, should probably be in a separate PR, but I'm not too bothered. It kind of goes with the rewrite… if you squint.
Do feel free to implement the setting in a separate PR, though. I'm just saying "no" to including it with this code rewrite, because adding a new setting is its own thing and should be handled as such.

The setting needs to be added into entity generation. A new pull request would need to be based on this rewrite, but I have no idea how to make that work in github. Cramming that into here seemed not too bad to me.

Two options, both starting with removing the added setting from this PR:

Wait until this PR is merged, then open a new one with the setting addition.
Open a new PR with the setting addition on top of this PR, then deal with any rebasing needed after this one is merged.

Speaking of rebasing, are you up for doing that to this PR? Or should I put together a cleaned-up history? I noticed you're using "Add files via upload", and rebasing can't be done through GitHub's web UI.

Since I can't do a rebase you'll have to do the cleanup and merge.

Squashed the code commits down, and removed the icon changes to a separate PR (#56) while I was at it.
This should be 100% good to go once I have a chance to fire up a quick test game of Factorio with this version of the mod to make sure I didn't break anything in the rebase.

The lines setting scale_info_icons = true are missing.
Either I uploaded the wrong file or the cleanup went wrong.

They're in the basic entity definitions, below the picture attributes.

Right-o, this tested out great.


CLN: Remove unused utility file

let's be honest, this is to improve our coverage stats

Ah, @danielballan this function is still used. Closing this PR


PostmarkError: Request body larger than maxBodyLength limit

When I try to send a batch of emails with sendEmailBatch() and one of the emails has a large body, then the sending of the whole batch fails and I get this error:
PostmarkError: Request body larger than maxBodyLength limit

This seems to be a known issue, it was discussed on StackOverflow. In my opinion this is a bug and can be fixed by setting maxBodyLength to Infinity. I did a pull request that fixes this problem. Let me know what you think of this.
Are there any downsides of adapting the maxBodyLength to Infinity?

hey @dcts ,
this update was released in latest version 2.7.1

hey @dcts ,
this update was released in latest version 2.7.1

Awesome thanks a lot!

Awesome thanks a lot!


Is pyads able to handle RPC methods and properties?

I'm working heavily with classes. My vision is to mirror a class on the PLC in a HMI without defining the class in python. As long as references (Interface, Reference To) aren't read, this should be possible. A property will be transformed into 2 RPC methods by the compiler.
__get<property name>:<TYPE>
__set<property name(TYPE)>
A class definition should be available in  the DATAINFO cmd of ADS. The reason for not wanting a definition in Python is, the info is already in place.

Unfortunately not directly using direct methods in pyads, this is planned, see #190.
However there will be a way of doing so by calling the CreateVariableHandle and ReadWrite functions yourself. This is just what the planned changes in #190 would do anyway.  You can follow the comment chain in #190 if you wish to implement this is your own program. If you do find the time and do this in your own application, please feel free to implement this feature into pyads, it will be appreciated!
Could you also please close this issue as it is effectively a duplicate of #190, any questions or comments, please feel free to comment on the issue #190.
Thanks

@rako233 can you please close this issue to help manage the issue tracker, thanks.

@rako233 @stlehmann this can be closed as duplicate of #190. Thanks.


mutate is not rerendering

Bug report
Description / Observed Behavior
I'm using mutate() to update data. Here is a little snip of what I do:
doRequest("someUrl", "PUT", someData).then(response => {
            mutate({}, false);
            mutate(response, false);
        });

this way it is working and  the data of my nextjs app is updated properly, but its pretty ugly since there are two re-renders triggered.
Here is what I want to do:
doRequest("someUrl", "PUT", someData).then(response => {
            mutate(response, false);
        });

or better would be
mutate(doRequest("someUrl", "PUT", someData), false);
This way data is not updated. I also tried to this:
doRequest("someUrl", "PUT", someData).then(response => {
            mutate({...response}, false);
        });

Changing  mutate(response, false); to  mutate({...response}, false); should not be nessesary since the object returned from server already is a new objectReference, but I gave it a try just to be sure.
Expected Behavior
I would expect swr to re-render every time when I use mutate with a new object reference.
Additional Context
SWR version: 0.3.4

Could it be that the data is the same? To avoid unnecessary request SWR compares the values deep equally and if they are the same it will not render again

No, the dates are similar, but not the same. At the point where the error occurred, it is about an object that has a list. All other values next to the list are the same, but the list has one element more or less.
{
_id: '',
name: ''
value: '',
someOtherValue: '',
children: []                          <- the length of this list is different
}

Having the same issue here

Could you provide a CodeSandbox to replicate it?

Yea I am having the same issue where mutate does not trigger a rerender when the object is nested. I thought mutate was gonna do a deep compare.

@troyacodes @EliasSchlatter could you provide a reproduction, like a CodeSandbox link? It looks like there's an object reference related issue.
I'd be happy to reopen this issue if anyone can help provide a reproduction. It just works fine for me personally.

@sergiodxa @shuding I don't think I can provide a code sandbox because the problem I'm running into seems to involve my .env file, and because though the variable is labeled PUBLIC_API_URI it is not actually a public API. Basically, none of my mutations ever re-render. Here's one piece of the exact code I'm using:
  const { data: plateData, error: plateError } = useSWR(
    [`${process.env.NEXT_PUBLIC_API_URI}/device/base/identifier/${id}`, jwt],
    fetcher,
  );

  const updatePlate = async (deviceToUpdate: BaseDevice) => {
    const resp = await fetch(
      `${process.env.NEXT_PUBLIC_API_URI}/device/base/id/` + deviceToUpdate.id,
      {
        method: 'PUT',
        headers: {
          Authorization: `Bearer ${jwt}`,
        },
        body: JSON.stringify(deviceToUpdate),
      },
    );
    const json = await resp?.json();
    if (json.id) {
      mutate(`${process.env.NEXT_PUBLIC_API_URI}/device/base/identifier/${id}`);
    }
  };

I've tried even cancelling the actual mutation and using mutate to insert the data myself, to no avail. It just does nothing. Am I doing something wrong with my URL structure? For what it's worth, the API URL resolves to an https:// URI, and I'm running the code locally, so perhaps that's the issue?

@cwbuecheler [${process.env.NEXT_PUBLIC_API_URI}/device/base/identifier/${id}, jwt] is an array key which will be serialized to sth different from the first string url. they key argument need to be consistent between mutate and useSWR if you want to manipulate it manullay.

@EliasSchlatter I feel it's the similar issue to #313 . check out https://github.com/vercel/swr/issues/313#issuecomment-638099853, this might help?

@huozhi - ah, got it. Makes sense that it'd need the entire array rather than just the string. Thanks!

Hi everybody,
sorry for my late response on this.
@huozhi thanks for mentioning #313 this post. However, I think there is a different problem here because the object that mutate is called with comes from the backend and should be a object reference in any case.  About the key argument thing, I'm not sure if I understood you right, but I'm using bound mutate so I think this is also not the source of my issue.
@sergiodxa @shuding here you can find a CodeSandbox

Having problems with useSWRInfinite and the bound mutate function. Trying to mutate array of arrays, which does not throw an error and logs the updated data correctly in console, but doesn't rerender the components. Anyone having the same problem?

@wobsoriano
Isn't this only revalidate on inside mutate()?


Update bitcoin-core.md

What
What is the reason of this change?
Following the guide here at this point, I ran the command systemctl status bitcoind.service and saw the following results and warning:
admin@raspibolt3:~/.bitcoin $ systemctl status bitcoind.service
* bitcoind.service - Bitcoin daemon
     Loaded: loaded (/etc/systemd/system/bitcoind.service; enabled; vendor preset: enabled)
     Active: active (running) since Thu 2021-12-30 23:15:37 EST; 5min ago
    Process: 786 ExecStart=/usr/local/bin/bitcoind -daemon -pid=/run/bitcoind/bitcoind.pid -conf=/home/bitcoin/.bitcoin/bitcoin.conf -datadir=/home/bitcoin/.bitcoin -startupnotify=chmod g>
   Main PID: 787 (bitcoind)
      Tasks: 17 (limit: 4165)
        CPU: 6.324s
     CGroup: /system.slice/bitcoind.service
             `-787 /usr/local/bin/bitcoind -daemon -pid=/run/bitcoind/bitcoind.pid -conf=/home/bitcoin/.bitcoin/bitcoin.conf -datadir=/home/bitcoin/.bitcoin -startupnotify=chmod g+r /home>

Warning: some journal files were not opened due to insufficient permissions.

Next I noted that the cookie had the correct read permissions on the group.
Next I failed to view the debug log:
admin@raspibolt3:~/.bitcoin $ tail -f /home/bitcoin/.bitcoin/debug.log
tail: cannot open '/home/bitcoin/.bitcoin/debug.log' for reading: Permission denied
tail: no files remaining

Next I noticed that not all the files in the ~/.bitcoin directory had the read permission for the bitcoin group.
I modified the group read permissions recursively on the directory with the command:
$ sudo chmod -R g+r /data/bitcoin

Then I was able to tail the log and the warning no longer appeared after restarting the bitcoind service.
Why
Why is this change important?
All users in the bitcoin group should be able to access all the files in the bitcoin data directory.
How
Recursively modifying the group read permissions on all files in the bitcoin data directory.
Scope

[ ] significant change to core configuration
[ ] independent bonus guide
[x] simple bug fix

Fixes #864
Test & maintenance
The changes can be tested by running the command and verifying that they can tail the debug.log file and no warning appears when inspecting the bitcoind service.
Animated GIF (optional)

Succeeded by #866


Unnecessary "web" guard is registered automatically after upgrading from Laravel v10 to Laravel v11

Laravel Version
11.2.0
PHP Version
8.3.4
Database Driver & Version
PostgreSQL v15.0
Description
It seems that "web" guard is registered automatically after upgrading Laravel from v10 to v11.
https://github.com/laravel/framework/blob/11.x/src/Illuminate/Foundation/Bootstrap/LoadConfiguration.php#L93-L103
https://github.com/laravel/framework/blob/11.x/config/auth.php#L39-L42
Our product only use custom guards, and we do not need "web" guard at all.
How to remove this "web" guard from config('auth.guards') entry?
Possible solution:
Change default guard config to empty.
https://github.com/laravel/framework/blob/11.x/config/auth.php#L39-L42
Steps To Reproduce

Create new Laravel project.
edit config/auth.php like this:

    'defaults' => [
        'guard' => env('AUTH_GUARD', 'web2'),
        'passwords' => env('AUTH_PASSWORD_BROKER', 'users'),
    ],
    'guards' => [
        'web2' => [
            'driver' => 'session',
            'provider' => 'users',
        ],
    ],

debug config('auth.guards') somewhere in the application

dd(config('auth.guards'));
// it will show 2 guards: "web", "web2"

Temporary workaround for this
Define "web" guard in project's config/auth.php even if we do not need one.
    'defaults' => [
        'guard' => env('AUTH_GUARD', 'web2'),
        'passwords' => env('AUTH_PASSWORD_BROKER', 'users'),
    ],
    'guards' => [
        'web' => [ // it will not be used anywhere in my application
            'driver' => 'session',
            'provider' => 'users',
        ],
        'web2' => [
            'driver' => 'session',
            'provider' => 'users',
        ],
    ],

https://laravel.com/docs/11.x/upgrade#application-structure This wouldn't been needed if the original configuration was kept during the upgrade.


Posible js concat semicolon problem fix

I experienced concat problem with this console error like this
Uncaugth TypeError: (intermadiate value)(intermadiate value)(...) is not a function

Thanks @orhanveli.


Cut down number of jobs in the presence of feature branches

https://github.com/Nemocas/AbstractAlgebra.jl/pull/805 created 3^4 = 81 jobs, because there are four packages, each with master, latest and we also had a custom branch with the same name in every repository (on purpose).
I know there are different ways to handle this, but one thing that came to mind was that if I am working on package A on branch #bla, and dependency B has also a branch #bla, then restrict the version for testing B only to B#bla (or maybe B#bla and B#stable) and not to all three B#latest, B#master and #Bla.
This is just one suggestion. Happy to hear what you think about this.

Agreed. It seems to me that the most relevant things to test are "all master", "all stable", "all matching branch (with fallback to master). Anything else is bonus and should be carefully selected

Yes, I like that suggestion. Maybe fallback to master and latest for the "all matching branch" case.

I implemented some extra features in #8:

it adds a global axis which is used for all pkgs that don't have an explicit branches entry to avoid the combinatorial explosion
testing matching branches is now done only when <matching> is specified in the branch-list, to make it more clear (can be used per pkg and globally). This will fall back to master per pkg when no matching branch is found.
(this means that if there is no matching branch for any other listed pkg it will do the same as specifying master)
there is a merge keyword for merging origin/master into the checked out projects (LibGit2 doesn't commit that merge by default so the directory will be dirty but that should not really matter for the CI)
allow running OscarDevTools with julia 1.3

I have also added some defaults for os, julia-version and branches in this repo here that will be used if nothing is specified in the toml (and allows updating these versions here instead of per repo):
const default_os = [ "ubuntu-latest" ]
const default_julia = [ "~1.6.0-0" ]
const default_branches = [ "<matching>", "release" ]

So with a rather empty config like here it will run just the following three jobs (no matching branch was found for Oscar and Singular):

Polymake#master,matching: [Oscar#master,Singular#master] - ubuntu-latest, julia ~1.6.0-0
Polymake#master,[Oscar,Singular]#release - ubuntu-latest, julia ~1.6.0-0
Nemo#master - macos-latest, julia ~1.6.0-0

Or for a AbstractAlgebra 805 insprired run running with this:

matching: [Nemo#wbhart#deprecate_coeffs,Oscar#wbhart#deprecate_coeffs,Hecke#wbhart#deprecate_coeffs,Singular#wbhart#deprecate_coeffs] - ubuntu-latest, julia ~1.6.0-0
[Nemo,Oscar,Hecke,Singular]#release - ubuntu-latest, julia ~1.6.0-0

Please ignore the test results of these jobs as they are not running with the corresponding branch in AbstractAlgebra but will just is that PR to detect the branch name for the other repositories ...

I like it.


Maybe we can

Because ’parameterTypes.length‘ It is a value in itself. If you write as before, it may reduce the clarity of the code！So I think it can be modified in this way！

This library is in maintenance mode. Only bugfixes will be merged.
In general, this changes the behavior though to query the length on every iteration. You can confirm this by looking at the change in Java bytecode. It's possible (or likely) that R8 would hoist the length lookup outside the loop, but I would have to confirm that and not everyone runs R8 on their apps.


Claims to update hosts but no sir

This is an awesome addition to VVV.. it speeds up my site creation from 5-10+min to ~1min.
However there seem to be an issue with writing to the hosts file.
It claims to do it in the log just like an ordinary vagrant up but it doesn't. It will also not ask for password even if it should  which I suspect is part of the reason why it fails.
To reproduce:

Setup a new site/project inside www with the vvv-init.sh, vvv-nginx.conf and vvv-hosts files.
Run flip set quick
Run vagrant provision

It will log things like
==> default:  * Added vvv.dev from /srv/www/vvv-hosts ==> default:  * Added local.wordpress.dev from /srv/www/vvv-hosts ==> default:  * Added local.wordpress-trunk.dev from /srv/www/vvv-hosts ==> default:  * Added src.wordpress-develop.dev from /srv/www/vvv-hosts ==> default:  * Added build.wordpress-develop.dev from /srv/www/vvv-hosts
and the new site but the new site will not be added to the hosts file if checked manually afterwards (it will also not work in the browser).
sudo nano /private/etc/hosts

If you do flip reset do you also get this issue?

No, running it with the regular script it works.. It's just when setting flip set quick


Issue 511: Raise expected proficiency level for the Secured Mission to Advanced + add callout to intro

resolves #511

@Ladicek WDYT?

Amended the note and added a link to the SSO Getting started guide.

LGTM, merging.


refactor: rename ModelViewSet class attr model to model_class

#51
Rename ModelViewSet class attr model to model_class

Hello,
Thank you so much for the feedback! I've run the tests and updated the PR. Let me know if there's anything else I missed.

Thank you so much for taking the time to help me and provide feedback! I really appreciate your expertise in understanding the problems I was facing and I learned a lot from this.

Thank you so much for taking the time to help me and provide feedback! I really appreciate your expertise in understanding the problems I was facing and I learned a lot from this.

You're very welcome @chengfangho, and thank you for your kind words!  Open source thrives on collaboration, and learning from each other is what makes it truly rewarding.
If you found our collaboration and the project valuable, please consider giving the repository a star 🌟 on GitHub. It's a simple way to support the project and helps us gain visibility within the community, allowing more people to benefit from our work.
Looking forward to future contributions, and feel free to reach out if you have any more questions or ideas. Happy coding!


[Portuguese] Localize 'Event-Driven Architecture'

Localize https://glossary.cncf.io/event-driven-architecture/

I would be happy to work on this content.

I would be happy to work on this content.

Great, thank you! I'll assign it to you, If you have any questions let us know at the slack channel :)


doesn't list the right files on a directory with multi-line filename.

[x] I have read through the manual page (man fzf)
[x] I have the latest version of fzf
[x] I have searched through the existing issues

Info

OS

[x] Linux
[ ] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[x] bash
[ ] zsh
[ ] fish

No i have not overwritten FZF_CTRL_T_COMMAND or $FZF_CTRL_T_OPTS
Problem
Invoking CTRL-T on a directory with multi-line filename, shows wrong number of filenames.
Steps to reproduce

touch "a file with newline"
rm CTRL-T

Screenshot

personal opinion
FZF must be able to select such filenames, it must present the escaped filename(like how it does when there's a space or other special character).
Having a multi-line filename is a rare scenario of course(with me it was an accident case of un closed shell quotes).

I'm aware of the limitation. But you can configure the command for CTRL-T to print items with null separator and fzf to read accordingly, for example:
FZF_CTRL_T_COMMAND='find . -print0'
FZF_CTRL_T_OPTS='--read0'

But we are not going to make this default since doing so will break any current user configuration that only sets one of the two.


update(pkg/cgo): increase MaxHandle

What type of PR is this?
/kind feature
Any specific area of the project related to this PR?
/area plugin-sdk
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
For https://github.com/falcosecurity/falco/issues/2074 point (B3), this increases the max number of cgo.Handles to 4096.
Does this PR introduce a user-facing change?:
NONE

Closing this, because this change is not really needed. In reality, the current value of 32 is enough for our current use cases.


ansible: dynamically pull vendor files from the secrets repo

This PR allows us to pull all vendor file secrets from https://github.com/AdoptOpenJDK/secrets/tree/master/vendor_files
Requirements

The user running the job has to have permission to clone the secrets repo
The user must have gpg access to the secrets. (see more information in the secrets repo readme)
The user must have dotgpg installed to extract the secrets

Thanks for those changes- So I'm trying to run this locally and I'm getting this error message (I'm running the playbook on a Ubuntu1604 Vagrant VM, connecting via the ssh port forward, so the host's file just contains '<IP_ADDRESS>:2222'
TASK [Get_Vendor_Files : check out AdoptOpenJDK/secrets] *********************************************************************************************************************************************
fatal: [<IP_ADDRESS>]: UNREACHABLE! => {"changed": false, "msg": "Failed to connect to the host via ssh: Warning: Permanently added '[localhost]:2222' (ECDSA) to the list of known hosts.\r\nwill@localhost: Permission denied (publickey,password).", "unreachable": true}

This is almost definitely an issue with how I've set it up, as opposed to the playbook, but I was wondering if you could shed some light?

yeah so you'll need to have dotpg and you github ssh key installed on that machine in order for it to work

Hmm, it's an issue with the way local_action is working:
TASK [Get_Vendor_Files : check out AdoptOpenJDK/secrets] *********************************************************************************************************************************************
task path: /home/will/Documents/vendor_files/ansible/playbooks/AdoptOpenJDK_Unix_Playbook/roles/Get_Vendor_Files/tasks/main.yml:6
<localhost> ESTABLISH SSH CONNECTION FOR USER: will
<localhost> SSH: EXEC ssh -F /dev/null -o ControlMaster=auto -o ControlPersist=60s -o StrictHostKeyChecking=no -o Port=2222 -o 'IdentityFile="/home/will/Documents/vendor_files/ansible/id_rsa"' -o KbdInteractiveAuthentication=no -o PreferredAuthentications=gssapi-with-mic,gssapi-keyex,hostbased,publickey -o PasswordAuthentication=no -o 'User="will"' -o ConnectTimeout=30 -o ControlPath=/home/will/.ansible/cp/0c5ff6558d localhost '/bin/sh -c '"'"'echo ~vagrant && sleep 0'"'"''
<localhost> (255, '', 'will@localhost: Permission denied (publickey,password).\r\n')

So it's connecting to will@localhost which is fine, except it's trying to connect to port 2222 which is currently assigned to the VM's ssh port.

@sxa can you confirm that this change also works for your setup? You're the other main person that regularly uses vendor_files to setup machines

Same linter failure as the other PR...

Merging as no objections from @sxa for 4 days. We can always back this change out if there is a problem

@gdams I think you may have done this while I was on vacation. This has caused a problem for the IBM team also using the playbooks. Pulling from the secrets repo absolutely shouldn't be the default operation - others should be able to run the playbooks as far as possible.

My understanding was that IBM skipped the AdoptOpenJDK tag??


Jupyter notebook not running code. Stuck on In [*]

Environment Information

Description
I just imported geemap to display maps, but the Jupyter kernel is still running. I waited for nearly half an hour, but it still didn't show up. I've rebooted jupyter many times and it's still not working. I don't know why the kernel keeps running.
The components are displayed when I run this code.
import ipyleaflet
m = ipyleaflet.Map()
m

What I Did

Are you using a VPN to access GEE?  See https://book.geemap.org/chapters/01_introduction.html#using-geemap-with-a-vpn

您是否使用 VPN 访问 GEE？请参阅https://book.geemap.org/chapters/01_introduction.html#using-geemap-with-a-vpn

yes,I've already set it up

No, your code does not show it is set up correctly. Try the following. If it does not work, it is not a geemap issue. It is your network issue.
import ee
ee.Authenticate()
ee.Initialize()

不，您的代码没有显示它已正确设置。尝试以下操作。如果它不起作用，则不是 geemap 问题。是你的网络问题。
import ee
ee.Authenticate()
ee.Initialize()

Thank you. I checked the port settings. I found out I had the wrong port


{AzureServiceBus} fixes Azure/azure-cli#25161 fix for migration abort issue

Azure/azure-cli#25161 fix for migration abort issue
Azure CLI command az servicebus migration abort fails with the below error:
  File "/opt/az/lib/python3.10/site-packages/azure/mgmt/servicebus/v2022_01_01_preview/operations/_migration_configs_operations.py", line 766, in revert
    request = build_revert_request(
  File "/opt/az/lib/python3.10/site-packages/azure/mgmt/servicebus/v2022_01_01_preview/operations/_migration_configs_operations.py", line 251, in build_revert_request
    "configName": _SERIALIZER.url("config_name", config_name, 'str'),
  File "/opt/az/lib/python3.10/site-packages/msrest/serialization.py", line 652, in url
    output = self.serialize_data(data, data_type, **kwargs)
  File "/opt/az/lib/python3.10/site-packages/msrest/serialization.py", line 760, in serialize_data
    raise ValueError("No value for given attribute")
ValueError: No value for given attribute

This is because the config-name parameter with ($defualt0 value isnt passed to the below function:
https://github.com/Azure/azure-sdk-for-python/blob/main/sdk/servicebus/azure-mgmt-servicebus/azure/mgmt/servicebus/v2022_01_01_preview/operations/_migration_configs_operations.py#L733
This PR fixes the above error by passing the config_name parameter to the revert function while performing migration abort operation.
Related command

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

AzureServiceBus


New Changes

added

Hi! Remember that WAHA Plus version has all this and more!
https://waha.devlike.pro/docs/how-to/waha-plus/

Hey, I'm sorry, I am not sure why It creates a PR to merge in your repo.


Allow snmpd to bind to multiple interfaces.

This patch makes it possible to bind snmpd to multiple interfaces other than ALL (for example to LAN and LAN CARP VIP). Binding to all interfaces is a security issue if you don't want to expose snmp to WAN.

RELENG_2_3_2 is a branch where we only push bug fixes and security fixes. This change is a new feature, please re-submit it to branch master


Add news preview to the homepage

Add the latest entry of the news (a single item) to the homepage in a simple div with a read more button.
Render it out using JS from the newest date of the news.json.
A nice to have feature, may clutter the homepage too much.

Not needed at this time, will clutter too much.


Resolve some problem

Already fixed (tested on my computer)
#445
Not test
#458
#367
#312
#269
and I do something change

Hey, thank you for your work but i can not accept these changes as they are right now. You are very welcome to fix stuff but this is a monster of a PR with over 1000 changed lines, including edits to a library file that should not be edited.
I would suggest that you find what exactly you want to fix, change only what you need to change and open a new PR describing what you've changed and why. I hope you understand that i can't manage huge PRs like this, simply look at the merge conflict list.

Also, Howdy does not officially support non-IR cameras. A config option like is_ir_camera therefore doesn't really fit.

OK, I will modify and resubmit
But I don’t recommend using tab as indentation, This does not comply with the PEP8 specification


Add jubileumsmiddagsgruppen

@89netraM merge my conflict!

Yö wtf did you actually merge the conflict? I'm scared

Ah looks like it was fixed


Ability to create own page in the shop

Is it possible to create your own page in the shop? In example I'd like to create http://devshop-xxx.shoparena.pl/appname/some-slug accessible by all users.
Examples:

plugin/app about page
some static pages
redirections for affiliation integration (which will set cookie, etc)
multi step administration for admins
some extra configuration for shop's users
etc

This question is not relevant to google-adwords-remarketing app. It should be addressed to DreamCommerce Appstore Team via technical support.
It is not possible to create your own "pages" in shop's domain. Each application's action is triggered via specific link in the shop and resulting html is embeded in iframe.


Question: Cross-compiling guidance

I understand that CoreRT can currently only be built with Visual Studio 2015. However, I am using Visual Studio 2017 RC for my projects, because of its updated support for .NET Core projects. I am hesitant to mix-and-match runtime support libraries (and, theoretically, the CoreRT output itself) built using VS2015's version of CL, and a manually written C++-based entry point built using VS2017's version of CL. Is this OK? If not, what is the best way to build only the native components using the VS2017 C++ compiler?
Furthermore, what is the best way to have multiple CoreRT builds, each targeting a different output architecture, sitting side-by-side and not interfering with each other? The reason I ask this is Microsoft's recent announcement about Windows 10 running on ARM. I would like to plan ahead for the day when I start compiling my apps for ARM, but at the same time don't want to break my existing Intel-based build process. What is the best way to accomplish this? Thanks!

Is this OK?

In theory, it should work. In practice, mixing different C/C++ compiler versions is sometimes painful

what is the best way to build only the native components using the VS2017 C++ compiler?

The build scripts does not have support for building with VS2017. We will be happy to take PR add such support. Since VS2017 is not released yet, it should be under a switch, e.g. build.cmd vs2017 will build using VS2017.
Here is similar change in CoreCLR repo that added VS2015 support some time ago to address the same problem: https://github.com/dotnet/coreclr/pull/822/

what is the best way to have multiple CoreRT builds, each targeting a different output architecture

The build output is set based on the output architecture, so having multiple CoreRT builds should work fine (modulo bugs): https://github.com/dotnet/corert/blob/6be9b2cd48c972ad21d89acc91f6756e6547e51f/buildscripts/buildvars-setup.cmd#L53
We have not done work yet to officially support cross-targeting, however we setting up everything to prepare for it. Somebody mentioned to me that it was not too hard to hack together a Windows-hosted Linux-targeting build, but the changes for that were not submitted to master.

I believe RyuJit is not able to cross-target if the pointer size differs between the host and target. So when/if you get closer to enabling cross-targeting we'll need to fix a bunch of assumptions in the code.


cmd/compile: Adding -gcflags "all=-N -l" does not add debugging information.

What version of Go are you using (go version)?

$ go version
go version devel go1.22-f7b4f02ba0 Fri Aug 4 15:41:19 2023 +0000 darwin/amd64

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE='auto'
GOARCH='amd64'
GOBIN=''
GOCACHE='/Users/aimuz/Library/Caches/go-build'
GOENV='/Users/aimuz/Library/Application Support/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFLAGS=' -mod=mod'
GOHOSTARCH='amd64'
GOHOSTOS='darwin'
GOINSECURE=''
GOMODCACHE='/Users/aimuz/go/pkg/mod'
GONOPROXY='gitee.com/wisecloud,gitee.com/wisecloud-archive'
GONOSUMDB='gitee.com/wisecloud,gitee.com/wisecloud-archive'
GOOS='darwin'
GOPATH='/Users/aimuz/go'
GOPRIVATE='gitee.com/wisecloud,gitee.com/wisecloud-archive'
GOPROXY='https://goproxy.dev,https://goproxy.io,https://goproxy.cn,direct'
GOROOT='/Users/aimuz/workspace/go'
GOSUMDB='sum.golang.google.cn'
GOTMPDIR=''
GOTOOLCHAIN='auto'
GOTOOLDIR='/Users/aimuz/workspace/go/pkg/tool/darwin_amd64'
GOVCS=''
GOVERSION='devel go1.22-f7b4f02ba0 Fri Aug 4 15:41:19 2023 +0000'
GCCGO='gccgo'
GOAMD64='v1'
AR='ar'
CC='cc'
CXX='c++'
CGO_ENABLED='1'
GOMOD='/Users/aimuz/workspace/go/src/go.mod'
GOWORK=''
CGO_CFLAGS='-O2 -g'
CGO_CPPFLAGS=''
CGO_CXXFLAGS='-O2 -g'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
PKG_CONFIG='pkg-config'
GOGCCFLAGS='-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -ffile-prefix-map=/var/folders/pt/qtn3qrvs6n9b_944ggd01nj00000gn/T/go-build4175838830=/tmp/go-build -gno-record-gcc-switches -fno-common'

What did you do?

Master branch compilations cannot be debugged using dlv.
Programs built with the -gcflags "all=-N -l" parameter cannot be debugged.
After investigation, the problem was caused by changes to https://go-review.googlesource.com/c/go/+/461697
When I change back to the historical version of the code, I can debug it properly
func DefaultPIE(goos, goarch string, isRace bool) bool {
	switch goos {
	case "android", "ios":
		return true
	case "windows":
		if isRace {
			// PIE is not supported with -race on windows;
			// see https://go.dev/cl/416174.
			return false
		}
		return true
	case "darwin":
		return goarch == "arm64"
	}
	return false
}

ld -v
@(#)PROGRAM:ld  PROJECT:dyld-1015.7
BUILD 16:59:22 Oct  1 2023
configured to support archs: armv6 armv7 armv7s arm64 arm64e arm64_32 i386 x86_64 x86_64h armv6m armv7k armv7m armv7em
will use ld-classic for: armv6 armv7 armv7s arm64_32 i386 armv6m armv7k armv7m armv7em
LTO support using: LLVM version 15.0.0 (static support for 29, runtime is 29)
TAPI support using: Apple TAPI version 15.0.0 (tapi-15<IP_ADDRESS>)
Library search paths:
Framework search paths:

What did you expect to see?
Be able to use dlv to debug programs properly
What did you see instead?
cannot be debugged

When I add the -ldflags "-buildmode=exe" parameter, he works fine!

Debug info is always included in the executable, -N -l just disables optimizations. Your problem is different: probably an just old version of delve, but it's hard to tell for sure because you haven't really described what's happening.

You are right, when I use the latest version of dlv he works fine, maybe older versions of dlv don't support PIE format executables?

It was probably this https://github.com/go-delve/delve/pull/3467

thank you


What is the benifit of ExtendableError?

What is the benifit of ExtendableError? why not let APIError class directly extend Error?

@MarkAdell i am wondering what direction you went since this project is stale and no one replied to your question?


Add a unit test expressly for AutoFilter descriptors

Some of the planned shared pointer slot refactoring is breaking AutoFilterDescriptor behaviors.  Because this type is fairly low-level, it makes sense to separate out tests dealing with this type into their own test case in order to speed debugging.

Old test looks unchanged, and the new ArityCheck test passes for me on Linux x64.


Enable Travis-CI for Julia 0.7 and nightly

Allow 0.7 and nightly fail.

Coverage remained the same at 100.0% when pulling 2a0c76dd497b1383ea0e91167dce516a62f1090d on travis into 0730f6495116eeab3977094e0f4978044aa00a01 on master.


[FEATURE] Add error bars capability

It would be nice to have the ability to add error bars around a dataset.
Here's a fork of the Chart.js library that claims to do this:
https://github.com/CAYdenberg/Chart.js-ErrorBars

I think this is a feature that is best implemented as a plugin since it is fairly specific

Closing since this will be best implemented as a plugin. https://github.com/chartjs/chartjs-chart-financial could be of some help

Could be that this is best implemented as a plugin, but I dont think this is a "fairly specific" requirement. If you measure something physical in the real world there will allways be an error. If you try to make some prediction of the future by looking at the past this prediction will allways have an error.  If you do some sort of statistics there will be an error too.


please change text for landing page?

Suggested text:
Welcome to PenguinWatch 2.0!
We now monitor over 100 sites! We need your help more than ever - to help monitor penguins, click "get started". If you are a school or educational site and you want to adopt and help us with a specific colony and get data feedbacks, please register (sign up) and contact us!

@penguintom79 This has been added in English now (thanks to @eatyourgreens). Ideally we'll want to add it in other languages too, assuming this will be a permanent change?

Nice! Where can I put the Czech translation for the new text? It is missing in the Translation page for PW. It would be good to add it there for other languages too. Thanks.

Hi @yshish The new text is showing up for me when I look at the translation page, under Todo. It was only added about 30 minutes ago, so maybe try reloading the page?

@eatyourgreens Oh, indeed! Thanks. I was looking under the Missing ones only 0:)
Going to translate it tonight.

Thanks!
Dr Tom Hart, Penguinologist.
Department of Zoology,
South Parks Road,
Oxford,
OX1 3PS.
+44 1865 600 170
www.penguinwatch.orghttp://www.penguinwatch.org/
www.penguinlifelines.orghttp://www.penguinlifelines.org/
From: eatyourgreens<EMAIL_ADDRESS>Sent: 06 April 2016 15:52
To: zooniverse/Penguins
Cc: Tom Hart
Subject: Re: [zooniverse/Penguins] please change text for landing page? (#86)
Hi @yshishhttps://github.com/yshish The new text is showing up for me when I look at the translation page, under Todo. It was only added about 30 minutes ago, so maybe try reloading the page?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHubhttps://github.com/zooniverse/Penguins/issues/86#issuecomment-206413240

May I ask what changes have been made to the Science page text that it appears under TODO as well? I haven't found any differences between the English and Czech versions, thanks.

BTW, the new text is ready to be published. I have already translated it.

Thanks @yshish New text should be on the site now. No changes to the Science page, as far as I know.

Then, shall I 'save' the translation for the Science page again, so it got away from the Todo? Or rather not?


[WIP] product search

Getting the following error:
1) Google\Cloud\Samples\Vision\productSearchTest::testImportProductSets
Google\ApiCore\ApiException: {
    "message": "Permission denied.",
    "code": 7,
    "status": "PERMISSION_DENIED",
    "details": []
}


Document Panel - Metadata UI

Feature Description
#405 was for the functionality of the Metadata. This ticket is a placeholder for when designs are added to figma.

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria
QA Instructions

@samitron7 Is there any update for UX this issue?