Add support for deferred SDL3 clipboard callbacks (and use it for images)

Alternative to / closes https://github.com/ppy/osu-framework/pull/6241

Improvements on previous PR

made in a generic way to support future expansions

e.g. for https://github.com/ppy/osu-framework/pull/6223

data is not copied when crossing the unmanaged boundary
SDL_HasClipboardData is checked before getting the data

How it works
All the required data for the clipboard callback is saved to a ClipboardCallbackContext object. An ObjectHandle is allocated for that object to keep it alive when it's owned by SDL (ownership is transferred upon successfully calling SDL_SetClipboardData).
When another application (or we) request clipboard data, the clipboard content is generated and the resulting ReadOnlyMemory<byte> is pinned (so the GC doesn't move it) and its pointer is passed to SDL.
When new clipboard data is set, our cleanup callback gets called so we unpin the ReadOnlyMemory and free the ObjectHandle.
Tested on

[x] Windows (with BMP format, still uses WindowsClipbard)
[x] macOS (still uses MacOSClipboard)
[x] X
[x] Wayland

Please make sure that TestSceneClipboard passes and that copying and pasting images to/from external apps also works.
iOS and Android are not supported (clipboard text only)

The Debug.Assert fails on my Linux machine.

This might not be due to not supporting the image format though. Here, dataCallback wrongly assumes that it'll only get called once with the requested mimetype, but on Linux it gets called multiple times with a different mimetype each time, even though we pass an array of mimetypes with only one element in it as an argument to SDL_SetClipboardData.
I'd recommend integrating the logic I wrote in my latest commit to solve this issue.
Apparently the image could be dropped at any time since we don't own it, so the only solution I see for this is to copy the Image itself in SetImage and pass that to the callback context.

Seems to fail for BMP files on Wayland:

[runtime] 2024-04-19 20:01:01 [verbose]: SDL error log [debug]: Pipe timeout
[runtime] 2024-04-19 20:01:01 [verbose]: Failed to get SDL clipboard data for image/png. SDL error: Pipe timeout
[runtime] 2024-04-19 20:01:01 [verbose]: SDL error log [debug]: Pipe timeout
[runtime] 2024-04-19 20:01:01 [verbose]: Failed to get SDL clipboard data for image/jpeg. SDL error: Pipe timeout
[runtime] 2024-04-19 20:01:01 [verbose]: SDL error log [debug]: Pipe timeout
[runtime] 2024-04-19 20:01:01 [verbose]: Failed to get SDL clipboard data for image/bmp. SDL error: Pipe timeout
[runtime] 2024-04-19 20:01:01 [verbose]: SDL error log [debug]: Pipe timeout
[runtime] 2024-04-19 20:01:01 [verbose]: Failed to get SDL clipboard data for image/tiff. SDL error: Pipe timeout
[runtime] 2024-04-19 20:01:01 [verbose]: SDL error log [debug]: Pipe timeout
[runtime] 2024-04-19 20:01:01 [verbose]: Failed to get SDL clipboard data for image/webp. SDL error: Pipe timeout
[runtime] 2024-04-19 20:01:01 [verbose]: SDL error log [debug]: Pipe timeout
[runtime] 2024-04-19 20:01:01 [verbose]: Failed to get SDL clipboard data for image/qoi. SDL error: Pipe timeout
[runtime] 2024-04-19 20:01:01 [error]: Step "retrieve image from clipboard" SingleStepButton triggered an error
[runtime] 2024-04-19 20:01:01 [error]: System.NullReferenceException: Object reference not set to an instance of an object.
[runtime] 2024-04-19 20:01:01 [error]: at osu.Framework.Tests.Visual.Platform.TestSceneClipboard.<TestImage>b__15_5() in /home/jannik/Development/osu-framework/osu.Framework.Tests/Visual/Platform/TestSceneClipboard.cs:line 73
[runtime] 2024-04-19 20:01:01 [error]: at osu.Framework.Testing.Drawables.Steps.SingleStepButton.<.ctor>b__1_0() in /home/jannik/Development/osu-framework/osu.Framework/Testing/Drawables/Steps/SingleStepButton.cs:line 19
[runtime] 2024-04-19 20:01:01 [error]: at osu.Framework.Testing.Drawables.Steps.StepButton.PerformStep(Boolean userTriggered) in /home/jannik/Development/osu-framework/osu.Framework/Testing/Drawables/Steps/StepButton.cs:line 124
[runtime] 2024-04-19 20:01:01 [error]: at osu.Framework.Testing.Drawables.Steps.StepButton.OnClick(ClickEvent e) in /home/jannik/Development/osu-framework/osu.Framework/Testing/Drawables/Steps/StepButton.cs:line 98
[runtime] 2024-04-19 20:01:01 [debug]: ClickEvent(Left) handled by "retrieve image from clipboard" SingleStepButton.
[runtime] 2024-04-19 20:01:01 [debug]: MouseClick handled by "retrieve image from clipboard" SingleStepButton.

Apparently the image could be dropped at any time since we don't own it, so the only solution I see for this is to copy the Image itself in SetImage and pass that to the callback context.

If we're copying the image, we're going to allocate memory (and might do some processing to change the pixel format) -- might as well save it to the clipboard directly.

If we're copying the image, we're going to allocate memory (and might do some processing to change the pixel format) -- might as well save it to the clipboard directly.

I think I was for whatever reason thinking about returning the image in multiple different formats at the same time, but I guess it's unnecessary...

That being said, it still fails for me. :(

@FreezyLemon your logs are a bit concerning. SDL_HasClipboardData is returning true, but then getting the data fails. Of course, it's not guaranteed that if SDL_HasClipboardData == true then SDL_GetClipboardData will always have some data (TOCTOU, etc.), but it failing in such a simple case warrants a bug report upstream.

That being said, it still fails for me. :(

Right, the callback can get called multiple times. SDL internally copies the data on windows.

@FreezyLemon your logs are a bit concerning. SDL_HasClipboardData is returning true, but then getting the data fails. Of course, it's not guaranteed that if SDL_HasClipboardData == true then SDL_GetClipboardData will always have some data (TOCTOU, etc.), but it failing in such a simple case warrants a bug report upstream.

The pipe timeout does feel like an upstream bug, yeah. I'll try and create a repro and then report it.

X11

Seems to work fine on X.

macOS should have full support for all clipboard data, but I'm mostly wondering which image formats are supported and compatible with other apps. Looking online, I see tiff, png and jpg mentioned.
We can always put multiple formats on the clipboard 😉
diff --git a/osu.Framework/Platform/MacOS/MacOSGameHost.cs b/osu.Framework/Platform/MacOS/MacOSGameHost.cs
index 0cb6f2362..45657c24a 100644
--- a/osu.Framework/Platform/MacOS/MacOSGameHost.cs
+++ b/osu.Framework/Platform/MacOS/MacOSGameHost.cs
@@ -11,6 +11,10 @@
 using osu.Framework.Input.Bindings;
 using osu.Framework.Input.Handlers;
 using osu.Framework.Input.Handlers.Mouse;
+using osu.Framework.Platform.SDL;
+using SixLabors.ImageSharp.Formats.Jpeg;
+using SixLabors.ImageSharp.Formats.Png;
+using SixLabors.ImageSharp.Formats.Tiff;

 namespace osu.Framework.Platform.MacOS
 {
@@ -35,7 +39,7 @@ public override IEnumerable<string> UserStoragePaths
             }
         }

-        protected override Clipboard CreateClipboard() => new MacOSClipboard();
+        protected override Clipboard CreateClipboard() => new SDL3Clipboard(TiffFormat.Instance /* or PngFormat, JpegFormat */);

         protected override ReadableKeyCombinationProvider CreateReadableKeyCombinationProvider() => new MacOSReadableKeyCombinationProvider();

On both Wayland and macOS, if I have something already in the clipboard, SetClipboardText("") doesn't clear it. The return code is 0 and there's no error in SDL_GetError().
SDL_ClearClipboardData() also seems to do nothing in this case, I assume that's because it's clearing data and not text.
Looks like an SDL issue.

On both Wayland and macOS, if I have something already in the clipboard, SetClipboardText("") doesn't clear it. The return code is 0 and there's no error in SDL_GetError().
SDL_ClearClipboardData() also seems to do nothing in this case, I assume that's because it's clearing data and not text.
Looks like an SDL issue.

Definitely an SDL issue as SetClipboardText() is just shorthand for SDL_SetClipboardData with text/plain mime type. Plus, calling SetClipboardText(NULL) or SetClipboardText("") actually calls SDL_ClearClipboardData()


weather fullfeatured

hi , i use your script weather full, it's very good, easy to use , but i get wrong information about the next sunrise or sunset , i don't know way, weather information is good .

Could you say what values you get and what you expect?

i don't get the same time for sunrise or sunset like on openweathermap.org , it's always show 11h16 and it's wrong it's should be 6:50 AM for example

Ah, found your problem.
This value displays the duration until the next sunset or sunrise.
But you could adapt that script to show the value as a time.

thnk's for help bro, im not programme i don't know how to do, if u can help please sir

Replace lines 79-85:
    if [ "$sun_rise" -gt "$now" ]; then
        daytime=" $(date --date="@$sun_rise" -u +%H:%M)"
    elif [ "$sun_set" -gt "$now" ]; then
        daytime=" $(date --date="@$sun_set" -u +%H:%M)"
    else
        daytime=" $(date --date="@$sun_rise" -u +%H:%M)"
    fi

@132ikl It indicates that we're providing a timestamp.

Yes, maybe. On the other hand, everyone can modify their own script as they like.

There's also people like OP who don't necessarily have the knowledge of how to do more complex scripting with bash, who would appreciate more options. Maybe there should be a standard to allow for small options (such as this issue or the one I implemented in info-twitch-countdown), similar to the way the boilerplate icon is handled

Yes, but it leads to a question: Where do we stop? We can not handle every case. Yes, there are a lot of nice ideas out there. But I want to keep things simple. The most scripts have only a few lines. I don't want to blow them up with many if-else conditions.
This repository is also already nearly foolproof. The reason for this is: There are no hundreds of options. There are simple scripts. That's all.
I knew this problem would occur. This is why README.md says the following:
This repository is not an exact blueprint. I guess every script has to be customized to make your Polybar unique.

That's fair. Maybe some remarkably modifiable scripts could have an "additions.txt" or something, which could include small modifications (such as those previously listed) users can make to customize their scripts. That may get out of hand as well, though, so that's simply an idea.


BlockReflection attribute inclusion for assemblies that are used on uap

@MichalStrehovsky commented on Thu Feb 09 2017
Framework assemblies that are used on netcore50 should include BlockReflectionAttribute=true in their project file unless there's a good reason not to.
It seems like something happened to framework reflection blocking with e.g. Microsoft.CSharp.dll in the 5.2.2 framework metapackage for UWP apps.
I was helping out troubleshooting a UWP failure where the dynamic keyword support goo was trying to reflect on the CSharpGetMemberBinder type (a private type in the assembly), and the customer was getting a MissingMetadataException on 5.2.2 metapackage, while things worked fine with the previous metapackage. The key difference seems to be the absence of the attribute. Removing reflection blocking is a breaking change.

@MichalStrehovsky commented on Thu Feb 09 2017
Cc @weshaggard @morganbr @davidwrighton @zamont

@jkotas commented on Thu Feb 09 2017
Did not we agree to make things well work without it? This stuff is super fragile...

@tarekgh commented on Fri Feb 10 2017
CC @AtsushiKan
@AtsushiKan is this something you can look at?

@MichalStrehovsky commented on Fri Feb 10 2017

Did not we agree to make things well work without it? This stuff is super fragile...

Getting this to work all the time would require a change in default RD.XML reflection policy on UWP apps from:
<Directives xmlns="http://schemas.microsoft.com/netfx/2013/01/metadata">
  <Application>
    <Assembly Name="*Application*" Dynamic="Required All" />
  </Application>
</Directives>

To:
<Directives xmlns="http://schemas.microsoft.com/netfx/2013/01/metadata">
  <Application Dynamic="All">
    <Assembly Name="*Application*" Dynamic="Required All" />
  </Application>
</Directives>

This change in policy comes with size on disk costs and compilation throughput costs. I don't remember an agreement about that.
This would still be breaking to apps that use the old form of the RD.XML (which is why the customer I was helping hit this by merely upgrading the framework metapackage). Upgrading framework metapackage should not be a breaking change.

@jkotas commented on Fri Feb 10 2017

Upgrading framework metapackage should not be a breaking change.

If this bug is for servicing existing framework package, I agree - servicing upgrades of the framework metapackages should be breaking change.
If this bug is to get this fixed in live master correctly, the framework package update will require matching toolset upgrade so I do not see a problem.
BlockReflectionAttribute have been used to communicate what types are part of the actual public surface vs. what types are internal implementation detail.
The same information is available in other form: The toolchain knows which assemblies and facades are part of the public surface and which ones are just internal implementation details. It can infer the reflection blocking form that. And the exceptions to this policy should be in rd.xml.

@morganbr commented on Mon Feb 13 2017
@MichalStrehovsky , that's a bit of an over-exaggeration. We've never had metadata for everything public in the framework, so we probably wouldn't add it for everything private in the framework.
@jkotas , having the toolchain do reflection blocking stuff internally doesn't sound too unreasonable, though it would still need some indication that an assembly should go through the process (some CoreFX assemblies do reflect on their internals).

@MichalStrehovsky commented on Mon Feb 13 2017

that's a bit of an over-exaggeration

As I wrote in the bug, the reason we hit this is because the DLR is reflecting on CSharpGetMemberBinder type in Microsoft.CSharp. That's a private type in Microsoft.CSharp. @jkotas suggested we make things work well without reflection blocking. How do you suggest we would fix it without the "over-exaggeration"?

@morganbr commented on Mon Feb 13 2017
@MichalStrehovsky , we have a few different tools for something like that:

Scoped rd.xml in the assembly. If the DLR is supposed to reflect on CSharpGetMemberBinder, we could add something like  to Microsoft.CSharp.dll (Altering that to make actual sense with what the DLR does, of course. Maybe SubTypes is more appropriate or something).
Better analysis. Maybe the DLR does something that Project N's analyzer should know about, but doesn't.
Worst case -- blanket rd.xml in the assembly. We can at least hold expansion to assemblies that are likely to do lots of reflection crawling as Microsoft.CSharp might without dragging in metadata for the internals of, say, the CoreRT type loader.

Of course, for this case if that type isn't even supposed to be visible for the DLR to function correctly, I'm all for reflection blocking.

@MichalStrehovsky commented on Mon Feb 13 2017
@morganbr Yes, and that will solve this for the exact problem the original problem was about and nothing else.
Let's say that nobody wants to do 1-3 for each private type a random app in the future might reflect on. We're left with either the RD.XML I proposed, or reflection blocking. Let's just stay at "we need to restore reflection blocking, period", seems like we agree on that :).

@morganbr commented on Mon Feb 13 2017
@MichalStrehovsky , that sounds perfectly reasonable. I was assuming there was something DLR-specific here, but I agree we don't want random reflection walks into private code.

@jkotas commented on Mon Feb 13 2017

If the DLR is supposed to reflect on CSharpGetMemberBinder, we could add something like to Microsoft.CSharp.dll

Yes, it is what I meant by "the exceptions to this policy should be in rd.xml".

@jkotas suggested we make things work well without reflection blocking

I meant to make reflection blocking work without corefx build magic that sprinkles BlockReflectionAttributes.

@tarekgh commented on Thu Mar 02 2017
I am moving this issue to corert according the conclusion in this discussion. if there is anything specific to Microsoft.CSharp need to be done we can open a new specific issue for that library.

Project N compiler work is not tracked on GitHub. @tarekgh Please open a TFS issue. Do not assign it to me.

@MichalStrehovsky could you please open the TFS bug as you have better context than me and you can mention what exactly need to be done there? I can open the bug and let you fill the details if you want to.

@tarekgh Just link to the original issue from CoreFX repo. It has all the details.

have logged https://devdiv.visualstudio.com/DefaultCollection/DevDiv/_workitems/edit/390112 feel free to edit or change anything there.


Token used to work but not anymore

Rasa NLU version 0.10.4:
Operating system : Ubuntu 16.04 LTS
Issue:
Token used to work in 0.9.3b but after installing Rasa_Core the Typing module changed and since then rasa_nlu stopped working giving error on ClassVar
So I upgraded the Rasa NLU to 0.10.4. The configuration is simple but when I use Token, and also pass it in URL it still gives me unauthorized output.
Thus I had to remove token from the configuration file to make it work. is it a bug?
This seems no brainer but can't figure out why token is not working. Please help.
config file
{"pipeline":"spacy_sklearn","num_threads":4,"path":"./models", "token":"12345"}

Which python version are you using? Can you please also provide an example request?

Hi, sorry for the delay as I was out of town without access to the Internet.
I am using Python 3.5
http://localhost:5000/status?token=12345
gives me unauthorized if I set token in above config file. This used to work earlier.

@pranav-nadkarni - @twerkmeister just created a PR #698, do you mind trying if that works for ou?

will try, and let you know. thanks

Yes this PR is working correctly. Thanks a lot !

Thanks tmbo & twerkmeister.


how do you extract the data?

hello,
I'm working on producing translation statistics for Fedora, and wonder how do you extract the list of translation files from Firefox, it's the only package for which I can't figure out how to automatically detect translation.
I hope your script could help me <3

answered in https://github.com/projectfluent/python-fluent/issues/117


BuildStream chokes when staging tarfiles with hardlinks that point outside of a chosen base directory

See original issue on GitLab
In GitLab by [Gitlab user @tlater] on Jun 24, 2019, 15:01
Summary
When BuildStream stages a tarfile, it will remove the first couple of characters from certain file names to extract things correctly. This is all fine and dandy, however, the code as-is, fails to extract certain kinds of hardlinks, as duly documented in the plugin itself.
Steps to reproduce

Create a tarfile with a hardlink that points outside its base directory.
Stage it.

What is the current bug behavior?
BuildStream reports a BUG and stops building.
What is the expected correct behavior?
BuildStream should continue building!
Relevant logs and/or screenshots
py37 develop-inst-noop: /home/tlater/Documents/Work/buildstream/buildstream
py37 installed: apipkg==1.5,arpy==1.1.1,astroid==2.1.0,atomicwrites==1.3.0,attrs==18.2.0,-e git+git[[Gitlab user @gitlab]](https://gitlab.com/gitlab).com:BuildStream/buildstream.git[[Gitlab user @f61f47cd519fa2402ff0b7d7e4a91ce63b84b0c3]](https://gitlab.com/f61f47cd519fa2402ff0b7d7e4a91ce63b84b0c3)#egg=BuildStream,Click==7.0,coverage==4.4,Cython==0.29.7,execnet==1.5.0,grpcio==1.17.1,isort==4.3.9,Jinja2==2.10,lazy-object-proxy==1.3.1,MarkupSafe==1.1.0,mccabe==0.6.1,more-itertools==6.0.0,pathlib2==2.3.3,pluggy==0.9.0,pluginbase==0.7,protobuf==3.6.1,psutil==5.4.8,py==1.8.0,pycodestyle==2.5.0,pyftpdlib==1.5.4,pylint==2.2.2,pyroaring==0.2.6,pytest==4.3.0,pytest-cov==2.6.1,pytest-datafiles==2.0,pytest-env==0.6.2,pytest-forked==1.0.2,pytest-timeout==1.3.3,pytest-xdist==1.26.1,ruamel.yaml==0.15.51,six==1.12.0,typed-ast==1.3.1,ujson==1.35,wrapt==1.11.1
py37 run-test-pre: PYTHONHASHSEED='2994009352'
py37 run-test: commands[0] | pytest --basetemp /home/tlater/Documents/Work/buildstream/buildstream/.tox/py37/tmp --cov=buildstream --cov-config .coveragerc --integration -m integration --no-cov -x --lf
============================= test session starts ==============================
platform linux -- Python 3.7.3, pytest-4.3.0, py-1.8.0, pluggy-0.9.0 -- /home/tlater/Documents/Work/buildstream/buildstream/.tox/py37/bin/python
cachedir: .tox/py37/.pytest_cache
rootdir: /home/tlater/Documents/Work/buildstream/buildstream, inifile: setup.cfg
plugins: xdist-1.26.1, timeout-1.3.3, forked-1.0.2, env-0.6.2, datafiles-2.0, cov-2.6.1
collecting ... collected 1292 items / 1291 deselected / 1 selected
run-last-failure: rerun previous 2 failures

tests/examples/autotools.py::test_autotools_build FAILED                 [100%]

=================================== FAILURES ===================================
_____________________________ test_autotools_build _____________________________

cli = <buildstream.testing.runcli.CliIntegration object at 0x7f9b78f73828>
datafiles = local('/home/tlater/Documents/Work/buildstream/buildstream/.tox/py37/tmp/test_autotools_build0')

    [[Gitlab user @pytest]](https://gitlab.com/pytest).mark.skipif(MACHINE_ARCH != 'x86-64',
                        reason='Examples are written for x86-64')
    [[Gitlab user @pytest]](https://gitlab.com/pytest).mark.skipif(not IS_LINUX or not HAVE_BWRAP, reason='Only available on linux with bubblewrap')
    [[Gitlab user @pytest]](https://gitlab.com/pytest).mark.datafiles(DATA_DIR)
    def test_autotools_build(cli, datafiles):
        project = str(datafiles)
        checkout = os.path.join(cli.directory, 'checkout')

        # Check that the project can be built correctly.
        result = cli.run(project=project, args=['build', 'hello.bst'])
>       result.assert_success()

tests/examples/autotools.py:29:
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _

self = <buildstream.testing.runcli.Result object at 0x7f9b78f7dda0>
fail_message = ''

    def assert_success(self, fail_message=''):
>       assert self.exit_code == 0, fail_message
E       AssertionError

src/buildstream/testing/runcli.py:115: AssertionError
----------------------------- Captured stdout call -----------------------------
BuildStream exited with code -1 for invocation:
	bst --no-colors --config /home/tlater/Documents/Work/buildstream/buildstream/.tox/py37/tmp/test_autotools_build0/cache/buildstream.conf --directory /home/tlater/Documents/Work/buildstream/buildstream/.tox/py37/tmp/test_autotools_build0 build hello.bst
Program stderr was:
[--:--:--][        ][    main:core activity                 ] START   Build
[--:--:--][        ][    main:core activity                 ] START   Loading elements
[00:00:00][        ][    main:core activity                 ] SUCCESS Loading elements
[--:--:--][        ][    main:core activity                 ] START   Resolving elements
[00:00:00][        ][    main:core activity                 ] SUCCESS Resolving elements
[--:--:--][        ][    main:core activity                 ] START   Resolving cached state
[00:00:00][        ][    main:core activity                 ] SUCCESS Resolving cached state
[--:--:--][        ][    main:core activity                 ] START   Checking sources
[00:00:00][        ][    main:core activity                 ] SUCCESS Checking sources

BuildStream Version 1.3.0+2355.gf61f47cd.dirty
  Session Start: Monday, 24-06-2019 at 13:40:01
  Project:       autotools (/home/tlater/Documents/Work/buildstream/buildstream/.tox/py37/tmp/test_autotools_build0)
  Targets:       hello.bst
  Cache Usage:   12K / None (0%)

User Configuration
  Configuration File:      /home/tlater/Documents/Work/buildstream/buildstream/.tox/py37/tmp/test_autotools_build0/cache/buildstream.conf
  Cache Directory:         /home/tlater/Documents/Work/buildstream/buildstream/integration-cache/cache-9kfotmgf
  Log Files:               /home/tlater/Documents/Work/buildstream/buildstream/.tox/py37/tmp/test_autotools_build0/cache/logs
  Source Mirrors:          /home/tlater/Documents/Work/buildstream/buildstream/integration-cache/sources
  Build Area:              /home/tlater/Documents/Work/buildstream/buildstream/integration-cache/cache-9kfotmgf/build
  Strict Build Plan:       Yes
  Maximum Fetch Tasks:     10
  Maximum Build Tasks:     4
  Maximum Push Tasks:      4
  Maximum Network Retries: 2

Pipeline
   buildable 7ea72715664f3125eac3063c5120c47ffe4327f90b61af0d809b6f153f617b49 base/freedesktop.bst
     waiting d098691a4af90e544427c18b3c051a738fc81558cb7b664287a43fa82a0fff61 base.bst
     waiting b4cda6988d11b1606e330a55aad1d2e9f1468b623e448bd09ae7e931f3e60d81 hello.bst
===============================================================================
[--:--:--][7ea72715][   fetch:base/freedesktop.bst          ] START   autotools/base-freedesktop/7ea72715-fetch.5780.log
[--:--:--][b4cda698][   fetch:hello.bst                     ] START   autotools/hello/b4cda698-fetch.5782.log
[00:00:00][b4cda698][   fetch:hello.bst                     ] SUCCESS autotools/hello/b4cda698-fetch.5782.log
[00:00:18][7ea72715][   fetch:base/freedesktop.bst          ] BUG     Fetch

    An unhandled exception occured:

    Traceback (most recent call last):
      File "/home/tlater/Documents/Work/buildstream/buildstream/src/buildstream/_scheduler/jobs/job.py", line 699, in child_action
        result = self.child_process()  # pylint: disable=assignment-from-no-return
      File "/home/tlater/Documents/Work/buildstream/buildstream/src/buildstream/_scheduler/jobs/elementjob.py", line 108, in child_process
        return self._action_cb(self._element)
      File "/home/tlater/Documents/Work/buildstream/buildstream/src/buildstream/_scheduler/queues/fetchqueue.py", line 45, in process
        element._fetch(fetch_original=self._fetch_original)
      File "/home/tlater/Documents/Work/buildstream/buildstream/src/buildstream/element.py", line 2183, in _fetch
        self.__cache_sources()
      File "/home/tlater/Documents/Work/buildstream/buildstream/src/buildstream/element.py", line 2932, in __cache_sources
        self.__sourcecache.commit(source, [])
      File "/home/tlater/Documents/Work/buildstream/buildstream/src/buildstream/_sourcecache.py", line 160, in commit
        source._stage(tmpdir)
      File "/home/tlater/Documents/Work/buildstream/buildstream/src/buildstream/source.py", line 740, in _stage
        self.stage(directory)
      File "/home/tlater/Documents/Work/buildstream/buildstream/src/buildstream/plugins/sources/tar.py", line 117, in stage
        tar.extractall(path=directory, members=self._extract_members(tar, base_dir))
      File "/home/tlater/Documents/Work/buildstream/buildstream/.tox/py37/lib/python3.7/tarfile.py", line 2002, in extractall
        numeric_owner=numeric_owner)
      File "/home/tlater/Documents/Work/buildstream/buildstream/.tox/py37/lib/python3.7/tarfile.py", line 2044, in extract
        numeric_owner=numeric_owner)
      File "/home/tlater/Documents/Work/buildstream/buildstream/.tox/py37/lib/python3.7/tarfile.py", line 2122, in _extract_member
        self.makelink(tarinfo, targetpath)
      File "/home/tlater/Documents/Work/buildstream/buildstream/.tox/py37/lib/python3.7/tarfile.py", line 2210, in makelink
        self._extract_member(self._find_link_target(tarinfo),
      File "/home/tlater/Documents/Work/buildstream/buildstream/.tox/py37/lib/python3.7/tarfile.py", line 2380, in _find_link_target
        raise KeyError("linkname %r not found" % linkname)
    KeyError: "linkname 'c/ssl/ct_log_list.cnf' not found"

[00:00:18][        ][    main:core activity                 ] FAILURE Build

Failure Summary
  base/freedesktop.bst:
    [00:00:18][7ea72715][   fetch:base/freedesktop.bst          ] BUG     Fetch

        An unhandled exception occured:

        Traceback (most recent call last):
          File "/home/tlater/Documents/Work/buildstream/buildstream/src/buildstream/_scheduler/jobs/job.py", line 699, in child_action
            result = self.child_process()  # pylint: disable=assignment-from-no-return
          File "/home/tlater/Documents/Work/buildstream/buildstream/src/buildstream/_scheduler/jobs/elementjob.py", line 108, in child_process
            return self._action_cb(self._element)
          File "/home/tlater/Documents/Work/buildstream/buildstream/src/buildstream/_scheduler/queues/fetchqueue.py", line 45, in process
            element._fetch(fetch_original=self._fetch_original)
          File "/home/tlater/Documents/Work/buildstream/buildstream/src/buildstream/element.py", line 2183, in _fetch
            self.__cache_sources()
          File "/home/tlater/Documents/Work/buildstream/buildstream/src/buildstream/element.py", line 2932, in __cache_sources
            self.__sourcecache.commit(source, [])
          File "/home/tlater/Documents/Work/buildstream/buildstream/src/buildstream/_sourcecache.py", line 160, in commit
            source._stage(tmpdir)
          File "/home/tlater/Documents/Work/buildstream/buildstream/src/buildstream/source.py", line 740, in _stage
            self.stage(directory)
          File "/home/tlater/Documents/Work/buildstream/buildstream/src/buildstream/plugins/sources/tar.py", line 117, in stage
            tar.extractall(path=directory, members=self._extract_members(tar, base_dir))
          File "/home/tlater/Documents/Work/buildstream/buildstream/.tox/py37/lib/python3.7/tarfile.py", line 2002, in extractall
            numeric_owner=numeric_owner)
          File "/home/tlater/Documents/Work/buildstream/buildstream/.tox/py37/lib/python3.7/tarfile.py", line 2044, in extract
            numeric_owner=numeric_owner)
          File "/home/tlater/Documents/Work/buildstream/buildstream/.tox/py37/lib/python3.7/tarfile.py", line 2122, in _extract_member
            self.makelink(tarinfo, targetpath)
          File "/home/tlater/Documents/Work/buildstream/buildstream/.tox/py37/lib/python3.7/tarfile.py", line 2210, in makelink
            self._extract_member(self._find_link_target(tarinfo),
          File "/home/tlater/Documents/Work/buildstream/buildstream/.tox/py37/lib/python3.7/tarfile.py", line 2380, in _find_link_target
            raise KeyError("linkname %r not found" % linkname)
        KeyError: "linkname 'c/ssl/ct_log_list.cnf' not found"

Pipeline Summary
  Total:       3
  Session:     3
  Fetch Queue: processed 1, skipped 1, failed 1
  Build Queue: processed 0, skipped 0, failed 0

=============================== warnings summary ===============================
.tox/py37/lib/python3.7/site-packages/pluginbase.py:439
  /home/tlater/Documents/Work/buildstream/buildstream/.tox/py37/lib/python3.7/site-packages/pluginbase.py:439: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated, and in 3.8 it will stop working
    fromlist, level)

-- Docs: https://docs.pytest.org/en/latest/warnings.html
WARNING: Coverage disabled via --no-cov switch!
========================== slowest 20 test durations ===========================
18.60s call     tests/examples/autotools.py::test_autotools_build
0.06s teardown tests/examples/autotools.py::test_autotools_build

(0.00 durations hidden.  Use -vv to show these durations.)
=========================== warnings summary (final) ===========================
.tox/py37/lib/python3.7/site-packages/pytest_cov/plugin.py:253
  /home/tlater/Documents/Work/buildstream/buildstream/.tox/py37/lib/python3.7/site-packages/pytest_cov/plugin.py:253: PytestWarning: Coverage disabled via --no-cov switch!
    warnings.warn(pytest.PytestWarning(message))

-- Docs: https://docs.pytest.org/en/latest/warnings.html
============ 1 failed, 1291 deselected, 2 warnings in 25.13 seconds ============
ERROR: InvocationError for command /home/tlater/Documents/Work/buildstream/buildstream/.tox/py37/bin/pytest --basetemp .tox/py37/tmp --cov=buildstream --cov-config .coveragerc --integration -m integration --no-cov -x --lf (exited with code 1)
___________________________________ summary ____________________________________
ERROR:   py37: commands failed

Other relevant information

BuildStream version affected: /milestone %BuildStream_v1.2 BuildStream_v2.0

In GitLab by [Gitlab user @tlater] on Jun 24, 2019, 15:02
I'm seeing this as part of !1413, so I will fix this before landing that.

In GitLab by [Gitlab user @tlater] on Jun 24, 2019, 15:07
changed the description

In GitLab by [Gitlab user @tlater] on Jun 24, 2019, 17:37
mentioned in commit 30a59b684b92bfe92c2470b5275cc63f0ebc10d7

In GitLab by [Gitlab user @tlater] on Jun 25, 2019, 15:03
mentioned in commit bc87b9c70891b5744d3456a225d65d9a354fc76b

In GitLab by [Gitlab user @tlater] on Jun 26, 2019, 18:20
mentioned in commit a14316f767f850fb1acde17138538cbe9f9be3dd

In GitLab by [Gitlab user @tlater] on Jun 26, 2019, 18:26
mentioned in merge request !1431

In GitLab by [Gitlab user @tlater] on Jun 27, 2019, 10:30
mentioned in commit 8dd64ea6ff7dbeacdd437dc1fc4193c2f92062c1

In GitLab by [Gitlab user @tlater] on Jun 27, 2019, 10:59
mentioned in commit 7e92a1243c0827c3cb5408dff81d0579f8e05cab

In GitLab by [Gitlab user @tlater] on Jun 28, 2019, 16:49
mentioned in commit 51a92d25053904af8094553965e038f681cd2377

In GitLab by [Gitlab user @tlater] on Jun 28, 2019, 16:54
mentioned in commit a2781fb48af5f92a00a3e24d305739fc715f6dd5

In GitLab by [Gitlab user @tlater] on Jul 2, 2019, 15:20
mentioned in commit 86a76c29c09d92e58280c5a9fc1736f5efe1e88f

In GitLab by [Gitlab user @marge-bot123] on Jul 2, 2019, 16:03
closed via merge request !1431

In GitLab by [Gitlab user @marge-bot123] on Jul 2, 2019, 16:03
mentioned in commit 2219874cdca548dd7f164ef3a0f77f5affeea2b0


Non-scoped plant adding fails with first selected location item

When using the add-plant form outside of a specific location view and the users selects the first item in the combobox then the creation fails.

Error does not occur anymore.


Update S3TransferUtility-Sample Swift

Update S3TransferUtility-Sample to use awsconfiguration.json
Update the README.md isntructions
Add a basic UI Test

Wouldn't it be better if we update the sample to use Amplify CLI instead of asking developers to update awsconfiguration.json?

Wouldn't it be better if we update the sample to use Amplify CLI instead of asking developers to update awsconfiguration.json?

Updated the instruction to use Amplify CLI and created a separate PR. Closing this one.


Instant OPD card and redirecting to user profile on confirmation of appointment

i have created OPD card that is containing the response data of the form
on confirming a appointment the user is redirected to User Profile .

@Luson045 please review my pr issue no #159

great work @17arindam add one more thing, let the user print it if he/she wants to, create a blob file and let the user print/download these details in pdf format. This way it can also be used in hospital OPD section too

@Luson045

please check my updates

@17arindam great 👍\


Fully support project, group and user avatars

This change set adds support for project, group and user avatars in the
same fashion this is already implemented for topics.
Currently it's not possible to remove avatars from those API - I'm
exploring possible solutions
here.
Nonetheless, I think this PR can already be merged.

Do you still need or want this? If so could you maybe cleanup the PR and reply to my questions? Then we can try to get it sorted...


refactor(connlib): implement new FFI guidelines

This updates connlib to follow the new guidelines described in #4262. I only made the bare-minimum changes to the clients. With these changes reconnect should only be called when the network interface actually changed, meaning clients have to be updated to reflect that.

@conectado @ReactorScram --
I believe you will want to implement similar logic as I did in #4133 in order to be smarter on the client side about calling reconnect and set_dns:

Only call reconnect if the primary interface has changed (or default egress interface)
Only call set_dns if the servers have changed

rebased to solve conflicts

Unfortunately this PR is now crashing the Apple clients again... :-(

@conectado Was there a reason why you dropped https://github.com/firezone/firezone/pull/4263/commits/2c2c6178cbfa9b5d6ed7ee69aef3727ba6ebced2 from this PR?

@conectado Was there a reason why you dropped 2c2c617 from this PR?

@thomaseizinger Not on pupose, seems to have been a mistake when I rebased


Main menu littel

Describe the bug
The main menu 3.4.7
Smartphone (please complete the following information):

Device: Nvidia Shield 64 bit]
Android Version: [e.g. Android 11]
App Version [e.g. 3.4.7]

To Reproduce
Steps to reproduce the behavior:

Go to '...main menu
Click on you see mobiel photon creen
Scroll down to no you can not use it
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots (Only if necessary)
Additional context
Add any other context about the problem here.

Hello!
This issue is caused by disabling landscape mode. I have now reenabled it.
This should be fixed in the lastest nightly version release (here)[https://github.com/zaneschepke/wgtunnel/releases/tag/nightly].
Once I have some more features/fixes made, I'll cut a new release to fdroid and google play.


docs: Make 0.6.0 docs main documentation

Makes the 0.6.0 docs the default docs. Does not remove the 0.5.0 docs to ensure we keep these archived. A follow up Pr on https://github.com/sidebase/docs is required and will be linked here.
Edit: Sister PR can be found here: https://github.com/sidebase/docs/pull/149

I'm a bit overwhelmed byu these changes: What is happening here? Why do we have to switch around all dirs and cannot just point latest to v0.6 and be happy?

I'm a bit overwhelmed byu these changes: What is happening here? Why do we have to switch around all dirs and cannot just point latest to v0.6 and be happy?

The problem is, that we had all the old links (on google, other pages etc.) never point to a version of the docs. If we were to leave them in that folder, we would need to update all the links, wait for google to update etc.
This would make a lot of problems. It would have been nicer if we from the very beginning had versioned paths, but as we did not do this, we now have this issue.

I see, but will this be resolved from now on / can we switch to the versioned approach now?

I see, but will this be resolved from now on / can we switch to the versioned approach now?

It would either:

Point all existing links and google searches (for a while until it updates) to a 404 (if we move the old docs into /v0.5)
Open outdated documentation if we keep the order as it is right now.

I don't think this is something we can tackle today.

point latest to 0.6

This is something we attempted during the initial versioning PR. The problem I ran into was that I could not create catch all redirects.
What this essentially means:

I would need to create a md file for every route we used to have in the docs and point each file to the new docs.

I would have loved to have a catch all route that says:

If link has nuxt-auth/X point to nuxt-auth/v0.6/X, but this is not possible.

See https://github.com/nuxt-themes/docus/issues/829 for the previous investigation.

Ok, and applying this solution without the redirect?

Ok, and applying this solution without the redirect?

This would result in many of our old links being inaccessible. This includes google search results until google updates them, which could take a while.

Added the banner as agreed:

Nit: It should probably be "You are viewing the docs for the newest version. [...]"

Nit: It should probably be "You are viewing the docs for the newest version. [...]"

True, good catch


[FLINK-8354] [flink-connectors] Add ability to access and provider Kafka headers

What is the purpose of the change
This changes allows to access Kafka headers in KeyedDeserializationSchema, and to provide Kafka headers via KeyedSerializationSchema.headers method
Brief change log

Add default headers method to KeyedSerializationSchema interface
Add default deserialize method with headers argument to KeyedDeserializationSchema
Add Kafka011Fetcher to get headers from Kafka ConsumerRecord and provide them as parameter KeyedDeserializationSchema
Modify FlinkKafkaProducer011 to request headers from KeyedSerializationSchema and add them to ProducerRecord.

Verifying this change
This change added tests and can be verified as follows:

*Added integration test Kafka011ITCase.testHeaders to verify that we can produce headers and consume them

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (yes)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (JavaDocs)

@yanghua, @aljoscha any other suggestions?

@alexeyt820 I think it is not easily possible to resolve the conflicts between this PR and #6105, which also touches the KeyedDeserializationSchema and follows a somewhat different approach.
I'm also not found of wrapping the Kafka record bytes into Record; that basically always ties deserialization of the bytes with access to other meta information, and makes it hard to reuse some already existing deserialization formats such as AvroDeserializationSchema.
Is it ok if I open a new PR that is based on this one and #6105, and see what I come up with? I might not be able to do that this week, but I can try next week.
cc @FredTing would that also be ok for you?

@tzulitai That's fine with me. Just let me know whether I should close PR #6105 now or wait until you submitted the new PR.

@tzulitai, using Record wrapping Kafka ConsumerRecord allows to add for example timestamp from PR #6105 w/o need to change client code, so it looks like more extensible approach. Not sure how it makes hard to reuse some already existing deserialization formats such as AvroDeserializationSchema, at least not harder then now - AvroDeserializationSchema will be wrapped via KeyedDeserializationSchemaWrapper in exactly same way as know. Also KeyedDeserializationSchemaWrapper calls only Record.value(), so it doesn't ties deserialization of byte with access to other metadata, not in terms of execution path (in logical terms it is always tied because underlying level - Kafka ConsumerRecord contains key, value and metadata)

For this, I would await the outcome of #6577. If we end up only having one "modern" Kafka connector we might just pass through the ConsumerRecord. Otherwise we might have to do some wrapping.

Rebased on top of #6703 and implemented KeyedDeserializationSchema.Record for Kafka 2.0 (had to keep KeyedDeserializationSchema.Record because KeyedDeserializationSchema is used by legacy connectors as well

@yanghua, @aljoscha is anything else need to be done with this PR ?

@alexeyt820 Please don't push any more changes right now. I'm having a look at this PR and might shuffle a few things around and rebase it.

@aljoscha, sure. Let me know if I can help

I created rebased PRs that have separate commits to be easier to review: #7781 and #7782

Closing because we have the other PRs.
Thanks for pushing this @alexeyt820!


No JAR-artifacts found

I run mvn package with success result, jar was created. But after clicking of refresh button i don't see any jars available.
DEBUG: Refresh list of JAR-artifacts in the project
DEBUG: Refresh JAR-artifact list.
INFO: No JAR-artifacts found.

I probably did not test the way you build the artefact. Could you try this sample app (following instruction in the wiki).
If it works - is it possible to get a sample app similar to one you faced issue with and an instruction how to reproduce it?

I have the same problem as @dementiev. It's easy to reproduce. Create a maven project with jar packaging and build the project. The plugin won't find the resulting jar.

I see. Let me look at this.

Could you check if the artifact appeared in the project out? If so - it is needed to hit a button "Refresh lits of Jar artifacts" to show it in the artifact list (I have a task to-do - to update it automatically).
If the project and a Jar-artifact is done differently in your case - could you describe steps, so I can reproduce it?
Thank you

The default output dir for maven is target.
Example:
mtest01/target/myjar-1.0-SNAPSHOT.jar

Artifact list is automatically refreshed after a project or artifact build - implemented in Pre-release 1.5.RC3

I think I managed to reproduce the issue properly now (image below). Seems I need to find the way to add this approach in addition to the implemented (and described above).
Thank you for your report.

Maven project artifacts supported - build an artifact with a Maven project and hit the button "Refresh list of JAR artifacts".
Please try this in the pre-release v1.5.RC4


Update the website README

Docsy has moved its user guide and changed its recommended Hugo version. And the page that we use as example for front matter has disappeared.

/assign


autorest generated client always requests for credentials

Describe the bug
We have generated apis without credentials, but the generated client always request for credentials.
config yaml:
input-file: swagger.json
typescript: true
output-folder: generated
Expected behavior
if we do not pass the --add-credential option, the client should be accessed without credentials.
Additional context
Add any other context about the problem here.
/**

@param credentials Subscription credentials which uniquely identify client subscription.
@param $host server parameter
@param options The parameter options
*/
constructor(
credentials: coreAuth.TokenCredential,
$host: string,
options?: WmsapiOptionalParams
) {
super(credentials, $host, options);
}

Seems the issues should be transferred to autorest.typescript as we have met this problem in TS.

Any update on this. I'm seeing it on clients I've been generating today, where we need that ability to access the API anonymously. Doing some testing, I get identical output whether I use the --add-credential option or not.

My command line is as follows - as per the documentation, there's no azure-arm option which I know triggers add-credential regardless of the flag setting
autorest --input-file=src/Tools.yaml --v3 --clear-output-folder --output-folder=src/clients/tools --typescript

@craignicol We have switched to openapi-generator


KUBE_TEST_REPO support

Upstream introduced a way to avoid having to set a config file via --e2e-repo-config and set each repo separately. Instead, if you set KUBE_TEST_REPO it will use the same registry for each to make it much simpler to use.
See https://github.com/kubernetes/kubernetes/commit/386f94ff032813ce15426b9ff9e7250281af9d8f

Probably just an --e2e-repo flag which takes a string. No validation needed, just sets an env var for the e2e plugin. Just needs documentation on the e2e page too. I'll make a documentation ticket for that.


Modernised TfIdf

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

56 of 56 (100.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 86.37%

Files with Coverage Reduction
New Missed Lines
%

lib/natural/brill_pos_tagger/lib/Corpus.js
1
74.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
9480

Relevant Lines:
10587

💛  - Coveralls


Hack Night Meeting

@jeremywight We decided to meet next week for a hack night.
The meeting is going to be focused on

Updating the API to allow for wait time estimates
Updating the API to add additional attractions like hotels and bars to the map
Hacking on Beer & Pizza codeforgvl.imaginethat.io/bpa

Can you send it out the Meetup invite for this?
Thanks,
-Adam

Replied to the other thread before I saw this. Will do, both meetup and OW
Calendar. Wish I could join you guys.
On Wed, May 18, 2016 at 12:12 PM, Adam Gautsch<EMAIL_ADDRESS>wrote:

@jeremywight https://github.com/jeremywight We decided to meet next
week for a hack night.
The meeting is going to be focused on

Updating the API to allow for wait time estimates
Updating the API to add additional attractions like hotels and bars
to the map
Hacking on Beer & Pizza codeforgvl.imaginethat.io/bpa

Can you send it out the Meetup invite for this?
Thanks,
-Adam
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/codeforgreenville/trolley-tracker/issues/146


added documentation for cript.utils

Description
set up documentation for utility functions of the CRIPT Python SDK
Changes

added Utility Functions documentation

added it to mkdocs.yaml
created an md file for it

wrote docstrings for load_nodes_from_json

Screenshot

@InnocentBug what are your thoughts on doing a schema validation of the node before converting it from JSON to Python node?

I think it may be a better idea to hold off on this for now until because we are working heavily on utils functions and there might be a lot of changes and updates that would make this outdated

@InnocentBug what are your thoughts on doing a schema validation of the node before converting it from JSON to Python node?

yeah, we should do that.

@InnocentBug wrote this a while back and I was thinking of holding off until later, but I'm thinking of just putting it into the validate-branch for now. Not sure if I should hold off or not.
I think if we keep putting things into the validate branch it might end up becoming giant and at the end create a log merge conflicts, but I am not sure if we should merge it into develop either because some of the serialization and deserialization code needs clean up. what are your thoughts?

@InnocentBug what are your thoughts on doing a schema validation of the node before converting it from JSON to Python node?

Would be great, but doesn't work.
A JSON from the backend does not conform with the schema.


feat(ansible): refactor PR #5053

Description
https://github.com/autowarefoundation/autoware/pull/5053
We can set up influxDB + Telegraf by following command:
./setup-dev-env.sh telegraf

Related links

Tests performed

Notes for reviewers

Interface changes

ROS Topic Changes

ROS Parameter Changes

Effects on system behavior

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[ ] I've confirmed the contribution guidelines.
[ ] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

Please check the pre-commit.ci - pr error. https://results.pre-commit.ci/run/github/41331761/1725851643.gTuOOOjpTPKPQ3PeonO-GA


feat: add unit test

Added unit testing based on regular expression code. Testing can cover relevant code.

@raedtwj
Hello, we believe that the support for regular expressions in your "json_exporter" feature is important to me. The reviewer has requested that you provide unit testing, documentation, and example usage, which we have added. I hope to accelerate the recommendation of this feature.

Added unit testing based on regular expression code. Testing can cover relevant code.

Thanks for the PR, appreciate your work

@raedtwj Hello, we believe that the support for regular expressions in your "json_exporter" feature is important to me. The reviewer has requested that you provide unit testing, documentation, and example usage, which we have added. I hope to accelerate the recommendation of this feature.

Thanks,  appreciate your work


(python3/php) python3 and php conflict due to depending on different vcredist versions

Expected Behavior

I expect that python3 and php can happily coexist when installed from chocolatey.
More technical: I expect python3 to use vcredist140 as a dependency instead of vcredist2015 (the other way around: PHP uses vcredist2015 instead of vcredist140 would be fine for me as well)
Current Behavior

python3 fails to upgrade due to a conflict. Python3 wants to downgrade vcredist140 to  14.26.28720.3 , but the PHP package doesn't declare itself compatible to that
Possible Solution

Use the same dependency in both packages. Whether that be vcredist140 for python3 or vcredist2015 for php is up to you.
Steps to Reproduce (for bugs)

Install php via chocolatey
End up with vcredist140 version  14.29.30037
Try to install python3
BOOM!

Context

Your Environment

Package Version used: python3 3.9.7, php 8.0.10
Operating System and version: Windows 11 21H2
Chocolatey version: 0.10.15
Install/uninstall gist: https://gist.github.com/mastacheata/4ffbbc7211f3a6f1d88d9ddd21691e48

cinst php
cinst python -i
"BOOM!"

Thanks for the hint, that does get around the problem indeed.
That's a workaround, but not a fix, though.
Is the underlying issue unfixable (i.e. php doesn't work with newer vcredist version) or did you just look for a fix that doesn't require changing anything?

I looked for a fix that doesn't change anything. The problem is specific to your usecase.

Is the underlying issue unfixable

In general case its IMO unfixable. In this case, its probably fixable - I essentially made python use php's vcredist (you could also go the other way around). This solution used the fact that python doesn't have any other dependencies so simple -i would suffice. If it depended on something else, it would require manual cinst of those.
Definite answer in each case depends on apps involved and more serious testing - I just tried if apps start but who knows what could break. In this case probably scanning original tool sites for answers is best idea.
CBB to look it up, but choco does support parallel packages in general, so you might look into that too. This was just quickest way to solve your problem.


Fix example disk type values for LKE (ext-4 vs ext4)

The Linode API supports either ext4 or raw for the disk type value, in LKE routes. The OpenAPI spec examples suggest ext-4 (with the -), which is incorrect. Requests using the ext-4 value yield a validation error response from the Linode API.
 \
     --header 'content-type: application/json' \
     --data '
{
  "k8s_version": "1.30",
  "label": "lkecluster12345",
  "node_pools": [
    {
      "autoscaler": {
        "enabled": true,
        "max": 12,
        "min": 3
      },
      "count": 6,
      "disks": [
        {
          "size": 1024,
          "type": "ext-4"
        }
      ],
      "labels": {
        "example.com/my-app": "teams"
      },
      "tags": [
        "example tag",
        "another example"
      ],
      "taints": [
        {
          "effect": "NoSchedule",
          "key": "example.com/my-app",
          "value": "teamA"
        },
        {
          "effect": "NoExecute",
          "key": "myapp.io/team",
          "value": "teamB"
        }
      ],
      "type": "g6-standard-4"
    }
  ],
  "region": "us-central",
  "tags": [
    "ecomm",
    "blogs"
  ]
}
'

{"errors": [{"reason": "Must be one of raw, ext4", "field": "node_pools[0].disks[0].type"}]}

It appears the ext-4 vs ext4 discrepancy was introduced in linode-api-docs/pull/283 (1 match for ext-4), and then disseminated in linode-api-docs/pull/956 (6 matches for ext-4).
This PR replaces all occurrences of ext-4 with the valid ext4, for LKE routes.

@Vernholio will take over introducing these changes from here - thanks!


bigger test suite

PR-Codex overview
This PR adds test cases for parsing and splitting text data in various formats. It includes tests for parsing lines, splitting text, extracting blob paths, and more.
Detailed summary

Added Lines class to parse lines from a full path
Added Split class to split text into an array of strings
Introduced BlobPath class to parse blob path from text
Included HashSplit class to split a path with a hash into a plain path

The following files were skipped due to too many changes: tests/ranged.test.ts

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@rultor merge

@h1alexbel OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@h1alexbel Done! FYI, the full log is here (took me 8min)


chore(main): release 1.0.0

:robot: I have created a release beep boop
1.0.0 (2023-11-14)
⚠ BREAKING CHANGES

add 'LIVEPEER_EXPORTER' prefix to environment variables (#22)

Features

add exporter log statements (db24ee1)
add livepeer-exporter (92309b4)
add reward call ratio (#17) (ff4d85d)

Bug Fixes

fix exporter port (c3fbb74)
fix reward and fee cut format (#14) (695683a)
fix reward call ratio calculation (#19) (c8e1bed)

Code Refactoring

add 'LIVEPEER_EXPORTER' prefix to environment variables (#22) (f92ad12)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/rickstaa/livepeer-exporter/releases/tag/v1.0.0 :sunflower:


Fixes the HTTP header issue with KBase's auth2 server

The problem was a newline in the access token added by a decoding step.
Closes #20

Actually, I'm going to merge this one into the test-coverage branch.


Detect Cycle in Undirected Graph

Given an undirected graph, The task is to check if there is a cycle in the given graph.
Description of Change

Checklist

[ ] Added file name matches File name guidelines
[ ] Added documentation so that the program is self-explanatory and educational - Doxygen guidelines
[ ] PR title follows semantic commit guidelines
[ ] Search previous suggestions before making a new one, as yours may be a duplicate.
[ ] I acknowledge that all my contributions will be made under the project's license.

Notes:

Dont know Why I'm getting error in Lint modified file in Awesome CI workflow. Please instruct me what should be done to fix it.

Can you please help me out in this?

Can you please help me out in this?

Sure. Let's discuss this on the Discord community. 🙂


refactor file loader and testcase loader

Coverage increased (+0.8%) to 80.662% when pulling c823777900dfca5ef79f6e5efd93949949fd8b05 on modular into b27a734eccae921ea6ccbbc00203da50ce708821 on master.

Coverage increased (+0.8%) to 80.662% when pulling c823777900dfca5ef79f6e5efd93949949fd8b05 on modular into b27a734eccae921ea6ccbbc00203da50ce708821 on master.

Coverage increased (+0.8%) to 80.662% when pulling c823777900dfca5ef79f6e5efd93949949fd8b05 on modular into b27a734eccae921ea6ccbbc00203da50ce708821 on master.


fix(KFLUXBUGS-1278): expose iib errors

This commit changes the add-fbc-contribution task to show the iib logs and to handle the InternalRequest pipelineRun always exiting cleanly. It instead checks the status for the exitCode.

e2e depends on https://github.com/hacbs-release/app-interface-deployments/pull/157. Once approved, I will merge https://github.com/hacbs-release/app-interface-deployments/pull/157, rerun e2e, then merge this. I will open a PR like https://github.com/hacbs-release/app-interface-deployments/pull/157 but for the stable branch when this is about to hit production

I think we should have at least one more test scenario of a task with a failed IR with exitCode=1 just like you did with the exitCode: 0.

done

/test release-pipelines-e2e-suite

/test release-pipelines-e2e-suite

/test release-pipelines-e2e-suite


When terraform refresh is executed, view grant is not loaded in tfstate.

Provider Version
0.25.30
Terraform Version
1.1.2
Describe the bug
When creating a view grant resource, no matter how many times I run terraform apply, I get a difference.
So, I run terraform state show after terraform refresh.
View grant is not loaded in tfstate.
Expected behavior
Once terraform apply is executed, no differences should be displayed.
Code samples and commands
resource "snowflake_view_grant" "grant" {
  database_name = "TEST_DB"
  schema_name   = "PUBLIC"
  view_name     = "TEST_VIEW"

  privilege = "select"
  roles     = ["TEST_ROLE"]

  with_grant_option = false
}

after running terraform apply, show tfstate.
$ terraform apply
...

$ terraform state show snowflake_view_grant.grant
# snowflake_view_grant.grant:
resource "snowflake_view_grant" "grant" {
    database_name     = "TEST_DB"
    id                = "TEST_DB|PUBLIC|TEST_VIEW|select|false"
    on_future         = false
    privilege         = "select"
    roles             = [
        "TEST_ROLE",
    ]
    schema_name       = "PUBLIC"
    shares            = []
    view_name         = "TEST_VIEW"
    with_grant_option = false
}

after running terraform refresh, show tfstate.
$ terraform refresh
snowflake_view_grant.grant: Refreshing state... [id=TEST_DB|PUBLIC|TEST_VIEW|select|false]

$ terraform state show snowflake_view_grant.grant
# snowflake_view_grant.grant:
resource "snowflake_view_grant" "grant" {
    database_name     = "TEST_DB"
    id                = "TEST_DB|PUBLIC|TEST_VIEW|select|false"
    on_future         = false
    privilege         = "select"
    roles             = []
    schema_name       = "PUBLIC"
    shares            = []
    view_name         = "TEST_VIEW"
    with_grant_option = false
}

Schema roles is empty.
Additional context

modify
privilege         = "select"

to
privilege         = "SELECT"

, no change was detected.


ALWAYS OPEN: add examples.

Add Tempy usage examples in the examples directory.
Ideally examples will contain a file for every TemPy usage and feature, plus a complete flask/weppy/whatever webapp made using TemPy.

Can I update the view for / so that it can link to the other examples?
As of now it simply says, Ready for some Tempy examples?, but it would be nice to have links to those examples right there.

Hi ! I'd like to contribute an example using chart JS.

Hi @maalbano yes thanks! Please make a pull request with your additions.

pull request here:  #106


Fix 113: [Defect] Inconsistent country and region code property names across addresses and admins themes

Simply moved definition of properties to root defs.yaml and updated places.yaml to reference it + updated examples

Would it make sense to instead change the names used in the admin layer to "region" and "country"?
If not, why not?
I went searching for example address schemas used by other groups, and I found this:
https://developers.google.com/search/docs/appearance/structured-data/local-business
which refers to the fields as "addressRegion" and "addressCountry".

My guess is that addresses can't be enforced to use ISO codes for country and region since they're always messy and have lots of exceptions. Maybe we leave this as-is for now and see if the address tf has a strong opinion as they form a better addressing schema?

My guess is that addresses can't be enforced to use ISO codes for country and region since they're always messy and have lots of exceptions. Maybe we leave this as-is for now and see if the address tf has a strong opinion as they form a better addressing schema?

Right. There's some notion that iso 3166-1 alpha 2 'can' be used but then you see that followed with examples using "USA". I think we should try to standardize to a single country reference in Overture. Most logical would be to use iso-3166-1 alpha 2 throughout, and call it like that.
Fixing that in the schema will become harder in the future though.

Places is already using iso 3166-1 codes for the country field, and iso 3166-2 codes for the region field, where available.  I agree that using standard codes is good.
However I think you can find a better way to communicate the format of the field, rather than encoding the format in the field name.  Perhaps you could add this as a comment, instead?
Part of the reason I am asking is that we have only just now (last week) gotten Microsoft to conform to the existing places schema.  If we ask them to change their format for the data they send us, it will create a lot of work for both me and them.  It sounds like the changes would not even be permanent since there is a separate "address group" that wants to rewrite this?


WSGIHeaderDict fails to get item if it's present and None in original dict

get key with not None value
>>> import bottle
>>> x = bottle.WSGIHeaderDict({"HTTP_X": "y", "HTTP_Z": None})
>>> print(x.get("x"))
y

get absent key
>>> print(x.get("w"))
None

get key with None value
>>> print(x.get("z"))
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib64/python3.6/_collections_abc.py", line 660, in get
    return self[key]
  File "/root/work/build/debug/config/vnc_openstack/.tox/py36/lib/python3.6/site-packages/bottle.py", line 2331, in __getitem__
    val = val.decode('utf8')
AttributeError: 'NoneType' object has no attribute 'decode'

Header values in WSGI environment dicts should never be None as far as I know. Empty headers are represented by an empty string and missing headers should not be in the environ dictionary at all. How did you end up with that situation? Maybe your WSGI server implementation or a middleware is buggy?

I'm also suspcting that WSGI headers can't have None values when they come to this code.
But here we have a sort of "Functional Unit Tests" which are not so simple and fails exactly on this issue. I've spent some time to dig into that legacy code and fails to find exact way how particular header is coming with None value.

In general (and it was a reason why I found it) - this breaks previous behaviour. With version<0.13 those UT work well. If I try same code with old version then I get None value for 'z' key.
Also I think that it's not good to have fail in the code which is caused by external data. I would prefer to check anything input data and raise exception consciouslyю
In terms of http headers - user may pass just a key without a value and it means that value is empty string. So it's also reasonable soluction.

this breaks previous behaviour.

Yes and no. Yes the behavior changed, but this is not a 'breaking change' because Bottle expects a WSGI compliant environment dictionary and if that expectation is not met, you enter 'undefined behavior' territory. Changing undefined behavior or fixing buggy behavior can happen anytime, even during patch releases. Relying on undefined behavior is actually a bug in itself and your tests revealed that bug.
This reminds me of https://xkcd.com/1172/. The old behavior was wrong, because returning None breaks the "all headers are strings" contract. Throwing KeyError breaks the dict API contract. Returning an empty string would lie about the presence of a header value where there is none. Crashing is actually the correct behavior in this situation.


instagram call checkpoint

please solve this issue as soon as possible

same problem

same issue

how to solve it

how do i fix this


Is there any way to listen to a plugin When it is enabled, disabled, installed or uninstalled?

Is there any way to listen to a plugin When it is enabled, disabled, installed or uninstalled?

It is Shutdown and Startup. Thank you~


Extract placeholder aspects from BasicPlaceholder

Fixes #77

@rultor merge

@rultor merge

@suseika OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@suseika Done! FYI, the full log is here (took me 1min)


Realm "Error" class confused with Swift 3's "Error" class

Goals
I have been trying to use Alamofire 4 with the latest version of Realm on Swift 3.
In Swift 3, when Alamofire requests fail, they return an "Error" object (Swift class).
I am using that error value to return a failure function as a completion handler with the error passed back.
The problem is that using that inside Realm makes Xcode confused between Realm's "Error" class and Swift's "Error" class!
Expected Results
If Realm's "Error" class was named differently, the code would run smoothly and the function would return a Swift "Error" class.
Actual Results
Xcode shows compile time errors (unable to compile) because the "Error" parameter is considered as a Realm "Error" class and therefore, what Alamofire is passing back is an invalid "Error" class.
Steps to Reproduce
The code below shows the compile time error that states that:

Cannot convert value of type 'Error' to expected argument type 'Realm.Error'

The code will work with Alamofire, Realm in Swift 3
Code Sample
extension Realm {
        func update<T:Object>(_ object:T, success:@escaping ()->Void, fail:@escaping (_ error:Error)->Void) {
            Alamofire.SessionManager.default.request("http://www.google.com", method: .get, encoding: URLEncoding.default)
                .responseJSON { response in
                    switch response.result {
                    case .success( _):
                        success()
                    case .failure(let error):
                        fail(error)
                    }
            }
        }
    }

Version of Realm and Tooling
Realm framework version: 2.7.0
Realm Object Server version: -
Xcode version: 8.2.1
iOS/OSX version: Mac OS X 10.12.4
Dependency manager + version: Cocoapods 1.1.1
Alamofire version: 4.4.0

Realm's error type is nested within the Realm class as Realm.Error. Since you're extending the Realm class, an unqualified reference to Error naturally refers to the type declared within the same scope. To refer to Swift's normal Error type within such a context, you'll want to explicitly qualify references to Swift.Error (that is, write Swift.Error rather than simply Error) .

I know that is possible. However, why does a Realm class have to conflict with a Swift class? I believe it might be better practice to not have such conflict. I have not seen a pod/library that creates their own String class and requires developers to call Swift.String instead because they're in the library's scope. It's just a matter of naming convention and I think it is unnecessary to have Realm.Error, maybe Realm.RealmError or Realm.RLMError would be better following the namings of RealmCollection or RLMSchema

Thanks for your feedback. The merits of naming our error type Error versus something else were discussed extensively during our transition to Swift 3, and we tried both variants in beta versions before settling on the current behavior. We have no plans to change our type names at this time.


Storage creating bucket name with env twice

Note: If your question is regarding the AWS Amplify Console service, please log it in the
official AWS Amplify Console forum

Which Category is your question related to?
adding s3 storage
Amplify CLI Version
4.6.0
What AWS Services are you utilizing?
Amplify CLI, s3
Provide additional details e.g. code snippets
When I type "amplify add storage" in my beta environment, follow the instructions, and enter an s3 bucket name of "test" (just an example) I end up with an s3 bucket named "testbeta-beta". My prod environment is similar - I get a bucket named "testprod-prod". Is it intentionally appending the environment name twice, or am I misunderstanding something? It seems like it would need to do it once, but I don't understand why it's happening twice. I'm not sure if this is a bug or not, or if its going to affect my ability to interact with these s3 buckets in the future.

@Chakron Ideally it shouldn't add the env names twice and I tested it on my local machine us-east-1 region and the CLI didn't add the env name twice.
Having said that, this shouldn't affect your app or it's usage if the  "aws_user_files_s3_bucket" set in the src/aws_exports.js file (in case of  JS projects) is set correctly to the right bucket.

Thanks for taking a look and trying to recreate the issue @kaustavghosh06. It's strange, this is a new project and I've just used the latest version of the Amplify CLI to create my resources. No idea why this is happening, but unless it becomes a problem in the future it doesn't look like I have too much to be concerned about other than the strange name. Maybe someone else will run into this and comment here.

We are facing the issue when creating new storage resource.

We are facing the same issue when creating new storage resource.


Gradient-Generator for frontend Developer

This website will help frontend developers to create custom gradient of their own choice or according to the business requirements.

Please add label : hacktoberfest-accepted


MS Cert thumbprint is not matching the actaul file downloaded for Visual Studio 2022

Description
The packer build fails as the downloaded installer for the Visual Studio signing certificate is not match the current download file
Platforms affected

[ ] Azure DevOps
[X] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] Ubuntu 24.04
[ ] macOS 11
[ ] macOS 12
[ ] macOS 13
[ ] macOS 13 Arm64
[ ] macOS 14
[ ] macOS 14 Arm64
[X] Windows Server 2019
[X] Windows Server 2022

Image version and build link
The signing cert Thumbprint of MS was wrong on the actual downloaded installer
Is it regression?
No
Expected behavior
Install Visual Studio 2022 using Packer should be working
Actual behavior
The packer build fails complaining that the downloaded installer not matching the Microsoft certificate thumprintthumbprint
Repro steps
on file images/windows/toolsets/toolset-2022.json change thumbprint to something and the build will fail

@1edevops Thank you for bringing this issue to us, we are investigating on this issue and will update you on this issue after our findings.

Hi @1edevops it will taken care by  https://github.com/actions/runner-images/issues/10201,
closing this issue, Thank you.


bar chart not found error

working in typescript environment. but after installing gifted chart package this error is showing

Hi Shubham,
Are you still facing this issue?
On Mon, Dec 20, 2021 at 3:22 PM shubhammaurya97 @.***>
wrote:

[image: image]
https://user-images.githubusercontent.com/82673986/146747721-37ceca0a-3fed-4a01-987f-3f16106b1281.png
working in typescript environment. but after installing gifted chart
package this error is showing
—
Reply to this email directly, view it on GitHub
https://github.com/Abhinandan-Kushwaha/react-native-gifted-charts/issues/50,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE5ERUCHQ64DPLWBLPLWSU3UR34GRANCNFSM5KNIM2YQ
.
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.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Graph for battery node percentage

Add graphical report for node battery percentage.

this feature is available in 0.0.2-alpha4


PR: Improve unfold_if_colapsed logic to consider nested blocks

Description of Changes
This PR unfolds all the nested code folding blocks when a search result lands inside of them.

[ ] Wrote at least one-line docstrings (for any new functions)
[ ] Added unit test(s) covering the changes (if testable)

[x] Included a screenshot or animation (if affecting the UI, see Licecap)

Issue(s) Resolved

Fixes #12485
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: @andfoy

@ccordoba12, the last commit should have addressed this last issue

Thanks for that @andfoy! One last thing though: is it possible to highlight the match after the region is centered? I think that's the last thing to make searches in folded regions work as they do in regular ones.

@ccordoba12, the rehighlight issue should be fixed on the latest commit:


[Question] Loc as the Difference Between Insertions and Deletions?

Are you sure that the loc is calculated from the difference between the insertions and deletions?
Not only my version of Git considers the loc the sum of insertions and deletions, also GitHub does.  For instance, go to the list of changed files from the latest Pull Request and hover the number on the left-hand side of src/main.rs.  (The README only had insertions, so that would be boring.)  GitHub shows to me then:  "45 changes: 24 additions & 21 deletions."
Did I misunderstand the purpose of the Loc struct or might that really be a candidate for a bugfix?

Loc stands for lines of code, so every commit can either increase or decrease the total number of lines of code.
The Loc struct was used to show the development of a software project over time. We choose the lines of code as a metric and used git to display them per day.
So no, this is not a bug.

Okay, thank you.
I am about to add some documentation to the code, I will add your explanation to the documentation.


Please add VirtualScroll for Slides

Note: for support questions, please use one of these channels:
https://forum.ionicframework.com/
http://ionicworldwide.herokuapp.com/
Short description of the problem:
Virtual Scroll is great. However, it works only with ion-list.
Can you please add this feature to ion-slides ?

Closing as a duplicate of https://github.com/driftyco/ionic/issues/6343


🚀 Feature: Indicate that the page is loading after a click

🔖 Feature description
When clicking on a link like the my shares link, the page is loaded in the background, I assume that is standard by Next.js and then displayed when the first request is finished and is only displayed then. This is fine but it makes the page seem sluggish and I have clicked a few times on some menus as I thought it was not registering the mouse click.
🎤 Pitch
I would prefer the browser or  the mouse to indicate that the browser is doing something in the back which would make the app appear faster even though it does not change the processing speed at all
e.g. when I load my "my shares" page, it currently takes about 7 seconds for the shares.js page

Are you running the Pingvin Share frontend in development mode? Can you share the command that you've used to start the frontend?

I believe its prod mode, I run the npm stuff directly without docker
npm install
npm run build
API_URL=http://localhost:8081
pm2 start --name="pingvin-share-frontend" .next/standalone/server.js

the backend server is running on port 8081


Cloudflare provider wants to delete and recreate record (without apparent changes), then fia

What happened?
I tried to run "pulumi up" such that cloudflare records would NOT be modified.
Pulumi still wants to delete and recreate all records.
Steps to reproduce
I have not been able to reproduce the problem; it does not appear on other stacks of the same projects. Stack state looks clean, but don't want to post it here because of too much sensitive information. I'm happy to provide parts of it on request.
Expected Behavior
No changes detected.
Actual Behavior
Changes have been detected, delete-replaces have been planned, which failed.
Versions used
CLI
Version      3.36.0
Go Version   go1.17.12
Go Compiler  gc
Plugins
NAME          VERSION
azure         5.12.0
azure-native  1.67.0
azuread       4.3.0
azuredevops   2.4.0
cloudflare    4.9.0
docker        3.2.0
dotnet        unknown
kubernetes    3.20.0
mysql         3.1.0
rabbitmq      3.2.0
random        4.8.0
tls           4.6.0
Host
OS       Microsoft Windows 11 Pro
Version  10.0.22000 Build 22000
Arch     x86_64
This project is written in dotnet: executable='C:\Program Files\dotnet\dotnet.exe' version='6.0.202'
Current Stack: prodstack
TYPE                                                                   URN
<< resources >>
Found no pending operations associated with prodstack
Backend
Name           DEVB
URL            file://.
User           DEVB\krali
Organizations
Dependencies:
NAME                           VERSION
Microsoft.AspNetCore.Identity  2.2.0
[redacted].Common              6.0.7868
Pulumi locates its logs in C:\Users\krali\AppData\Local\Temp by default
Additional context
When asking for details, it does not show any apparent changes:
Do you want to perform this update? details
  pulumi:pulumi:Stack: (same)
    [urn=urn:pulumi:prodstack::[redacted]::pulumi:pulumi:Stack::[redacted]-prodstack]
                --cloudflare:index/record:Record: (delete-replaced)
                    [id=[redacted - correct record id]]
                    [urn=urn:pulumi:prodstack::[redacted]::pulumi:providers:kubernetes$k8sx:external-ingresscontroller$cfx:registration$cloudflare:index/record:Record::cf-[redacted].[redacted].net-record]
                    [provider=urn:pulumi:prodstack::[redacted]::pulumi:providers:cloudflare::cloudflare-[redacted]::b940bcb5-c342-4881-ae13-2e5ebda4ef6c]
                +-cloudflare:index/record:Record: (replace)
                    [id=[redacted - correct record id]]
                    [urn=urn:pulumi:prodstack::[redacted]::pulumi:providers:kubernetes$k8sx:external-ingresscontroller$cfx:registration$cloudflare:index/record:Record::cf-[redacted].[redacted].net-record]
                    [provider=urn:pulumi:prodstack::[redacted]::pulumi:providers:cloudflare::cloudflare-[redacted]::b940bcb5-c342-4881-ae13-2e5ebda4ef6c]
                  ~ allowOverwrite: false => false
                  ~ name          : "[redacted]" => "[redacted]"
                  ~ proxied       : true => true
                  ~ type          : "CNAME" => "CNAME"
                  ~ value         : "[redacted].[redacted].net" => "[redacted].[redacted].net"
                  ~ zoneId        : "[redacted - correct zone id]" => "[redacted - correct zone id]"
                ++cloudflare:index/record:Record: (create-replacement)
                    [id=[redacted - correct record id]]
                    [urn=urn:pulumi:prodstack::[redacted]::pulumi:providers:kubernetes$k8sx:external-ingresscontroller$cfx:registration$cloudflare:index/record:Record::cf-[redacted].[redacted].net-record]
                    [provider=urn:pulumi:prodstack::[redacted]::pulumi:providers:cloudflare::cloudflare-[redacted]::b940bcb5-c342-4881-ae13-2e5ebda4ef6c]
                  ~ allowOverwrite: false => false
                  ~ name          : "[redacted]" => "[redacted]"
                  ~ proxied       : true => true
                  ~ type          : "CNAME" => "CNAME"
                  ~ value         : "[redacted].[redacted].net" => "[redacted].[redacted].net"
                  ~ zoneId        : "[redacted - correct zone id]" => "[redacted - correct zone id]"

("true" and "false" are red/green here)
I really don't want to do this because this is production. As a last resort, however, I've tried it on a less important service and it still failed:

Do you want to perform this update? yes

Updating (prodstack):
     Type                                   Name                               Status                   Info
     pulumi:pulumi:Stack                    [redacted]-prodstack                        **failed**               1 error
     └─ pulumi:providers:kubernetes         flatten-k8s-PROD                                            [diff: ~version]
        └─ k8sx:external-ingresscontroller  backend
           └─ cfx:registration              cf-[redacted].[redacted].net
 +-           └─ cloudflare:index:Record    cf-[redacted].[redacted].net-record  **replacing failed**     1 error

Diagnostics:
  pulumi:pulumi:Stack ([redacted]-prodstack):
    error: update failed

  cloudflare:index:Record (cf-[redacted].[redacted].net-record):
    error: deleting urn:pulumi:prodstack::[redacted]::pulumi:providers:kubernetes$k8sx:external-ingresscontroller$cfx:registration$cloudflare:index/record:Record::cf-[redacted].[redacted].net-record: 1 error occurred:
        * error deleting Cloudflare Record: HTTP status 400: Could not route to /zones/dns_records/[redacted - correct record id], perhaps your object identifier is invalid? (7003), No route for that URI (7000)

Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

Downgrading Pulumi.Cloudflare to 3.5.0 seems to fix the problem. Moving back up to 4.0.0 causes the issue to reappear.

Hi @kralikba
Please can you try and provide a set of code that helps me to reproduce the problem?
Paul

Hi @kralikba
Please can you try and provide a set of code that helps me to reproduce the problem?
Paul

I have not yet been able to reproduce the issue on a minimal part of the system.
Do you have any idea how the zoneId could become empty/null in the request URL? (as per the last line of pulumi's output in my error description). Adding a debugging proxy between the provider and the cloudflare web service (using Fiddler) confirms that somehow the request path is wrong. Even more, it is shown to be /zones//dns_records/[redacted - correct record id] (note the //, between which the zone id should be seen).

same issue here: https://github.com/pulumi/pulumi-cloudflare/issues/325

Since no repro was originally provided, I'll assume it's a duplicate of #325


fix: checkbox needs to be touched before error message

Summary
Type

Bug

Summarise concisely:
What is expected?
Checkbox should not be showing error message directly
The following changes where made:
(Describe what you did)

add touched before showing error

Codecov Report

Merging #173 (2d1b563) into main (f65292f) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #173   +/-   ##
=======================================
  Coverage   94.44%   94.44%
=======================================
  Files          21       21
  Lines         234      234
  Branches       25       25
=======================================
  Hits          221      221
  Misses          9        9
  Partials        4        4

Impacted Files
Coverage Δ

src/components/CheckboxField/index.tsx
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


chore: fix invalid release check + docs update

Codecov Report

Merging #32 (1e24129) into next (1eac3d8) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              next       #32   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           23        23
  Lines          230       230
  Branches        11        11
=========================================
  Hits           230       230

Continue to review full report at Codecov.

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


[FEAT] API Endpoint to Fetch all Roles within an Organisation.

Description
Develop an API endpoint to fetch all roles within the application. This endpoint will allow retrieving a list of all existing roles and their details. Roles define sets of permissions that can be assigned to users.
Acceptance Criteria
Endpoint: The endpoint /api/v1/roles/all should be accessible via a GET request.
Successful Retrieval: When the endpoint is accessed, it returns a 200 OK status with a list of all roles and their details.
Server Error: If an unexpected server error occurs, the endpoint returns a 500 Internal Server Error status with an appropriate error message.
Purpose
The Fetch All Roles endpoint allows for retrieving all roles in the system. This is essential for managing user permissions and access levels within the application.
Requirements
Fetch All Roles Endpoint: Create an endpoint /api/v1/roles/all to handle the retrieval of all roles.
Return Detailed Information: Return detailed information on each role.
Handle Errors: Implement error handling for server errors.
Expected Outcome
The endpoint should successfully retrieve and return all roles, ensuring clear and effective role management within the system. This allows for the definition and review of the sets of permissions that can be assigned to user roles.
Endpoint URL
GET /api/v1/roles/all
Success Response
Code: 200 Okay
{
  "status_code": 200,
  "data": [
    {
      "id": "role-id-1",
      "name": "manager",
      "description": "The manager role encompasses overseeing team operations, managing resources, and ensuring the achievement of project goals. Managers are responsible for strategic planning, team leadership, performance evaluations, and communication with upper management.",
      "created_at": "DateTime"
    },
    {
      "id": "role-id-2",
      "name": "admin",
      "description": "The admin role involves overseeing the administrative tasks and managing user accounts.",
      "created_at": "DateTime"
    }
  ]
}

Server Error
Code: 500 INTERNAL SERVER ERROR
{
  "status_code": 500,
  "message": "An unexpected error occurred, roles not retrieved"
}

Testing
Test Scenarios

Successful Retrieval
Confirm that the status code is 200.
Verify that the response body contains the correct details of all roles.

Server Error Handling

Simulate an unexpected error during the roles retrieval process.
Verify that the endpoint returns a 500 Internal Server Error status code.
Confirm that the response body contains the appropriate error message.
Testing Role Retrieval
curl -X GET http://localhost:8000/api/v1/roles/all

In the update and fetch roles ticket, we have a description, don't we?

@highb33kay DEscription updated

Have you checked for duplicate?

@highb33kay Changed, thanks Chairman

I'm not exactly sure why this ticket is useful, what inspired this ticket?

Endpoint to retrieve roles
Endpoint to create permissions
Endpoint to retrieve permissions
Endpoint to update user's role from implementing users roles and permissions

@nedssoft This one https://github.com/hngprojects/hng_boilerplate_nestjs/issues/259


mdbook-pdf: init at 0.1.2

Signed-off-by: Hollow Man<EMAIL_ADDRESS>Description of changes
mdbook-pdf is A backend for mdBook written in Rust for generating PDFbased on headless chrome and Chrome DevTools Protocol.
crates.io link: https://crates.io/crates/mdbook-pdf

Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Thank you!


sf::Packet::operator>>(char*) is unsafe

The title says it all, this method (plus its wchar_t equivalent) is unsafe, as there's no way to specify the size of the buffer.
I think it should be made something like sf::Packet::read(char* buffer, size_t size) instead. Should someone want a buffer overflow, the old behavior can still be gotten by making a call like packet.read(buffer, -1).

This task will probably be part of #112.


Refactor EndpointSet

This commit refactors the EndpointSet struct in order to make it easier
to understand and work with.
Signed-off-by: Filip Petkovski<EMAIL_ADDRESS>
[ ] I added CHANGELOG entry for this change.
[x] Change is not relevant to the end user.

Changes

Refactor endpointset.go.
Add additional tests to help understand specific behavior of EndpointSet.

Verification
Unit tests and manual testing.

cc @saswatamcode

cc @SrushtiSapkale for review as @fpetkovski is refactoring code we will be using in your project (:


Use docker-compose pull to update local images

You can use docker-compose pull command to update local images instead of upgrading them by name.

Nice! thanks!


Apply latest token ds-auro-icon-size #28

Alaska Airlines Pull Request
Type of change:
Please delete options that are not relevant.

[ ] New capability
[x] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team
Summary by Sourcery
Apply the latest token for icon size by updating the variable to --ds-auro-icon-size and update version numbers in related files.
Enhancements:

Update icon size variable to use --ds-auro-icon-size for consistency.

Chores:

Update version numbers in buttonVersion.js and iconVersion.js files.

:tada: This PR is included in version 2.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add runtime library dependency by gradle plugin for multiplatform modules

Currently implementation 'com.github.sergeshustoff.dikt:dikt-runtime:' is not added by plugin to multiplatform modules. Seems that if works fine others

Done in 1.0.0-alpha5

For some reason runtime library is added when gradle plugin applied old-school way with
classpath "io.github.sergeshustoff.dikt:dikt-gradle-plugin:1.0.0-alpha5"

in buildscript.dependencies. But it's not added when gradle plugin is applied in module via new plugin api like this:
plugins {
    id "io.github.sergeshustoff.dikt" version "1.0.0-alpha5"
}

fixed in 1.0.0-alpha6


Add a max_size field to the TCP source config parser syntax.

This PR adds an additional field to TCPSourceConfigCLIParser such that you can specify a maximum message size per source. The format is either source_name@host:service or source_name@host:service:max_size.  This should be backward compatible to existing applications.
For example, an app might have a command line like this:
    --in foo-source@<IP_ADDRESS>:7100,bar-source@<IP_ADDRESS>:7102:200000000

where foo-source has the default size of 16384 and bar-source 200 MB.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


feat: add pre-commit hooks

Add a simple playbook that can install pre-commits and register a few simple hooks. This process is implemented as a hook that will run post control host bootstrap.

check-yaml: perform basic yaml syntax linting
end-of-file-fixer: identify and automatically fix missing newline
trailing-whitespace: identify and automatically fix excessive white space
ripsecrets: identify and prevent secrets from being committed to the branch

Note: this is currently configured as an opt-in feature and will only install and register hooks if enable_pre_commit_hooks is set to true in install-pre-commit-hooks.yml

How are we expecting this to be used? Is it something that would be enabled for everyone in a given Kayobe config, or opted into per-person?

I think having it is as an opt-in at least within SKC it the sensible move. If individual engineers want to enable it within their own checkout of the site config they can do and in situations where their is a single checkout shared enabling it would a group decision.

@jackhodgkiss should we get this merged? I'm happy to rebase it on antelope for you if you're busy


[Bug]: Search Bar Not Working on the Website

Describe the bug

The search bar on the website is not functioning. When attempting to use it, no results are returned, and it appears unresponsive.

To Reproduce
Steps to reproduce the behavior:

Go to the website's homepage.
Click on the search bar.
Enter any search term.
Press the search button or hit enter.
Observe that no results are returned and the search bar appears unresponsive.

Expected behavior.

The search bar should return relevant results based on the entered search term. It should be responsive and display a list of results matching the query.

Screenshots
https://github.com/user-attachments/assets/b916b0e0-180e-43ca-a6d9-3fe62da74894
Desktop (please complete the following information):

OS: Windows
Browser: Chrome
Version: 127.0.6533.72 (Official Build) (64-bit)

Smartphone (please complete the following information):

Device: [e.g. iPhone12]
OS: iOS 17
Browser: safari
Version : 17.5

Additional context
Add any other context about the problem here.

@4darsh-Dev Thanks, I will start working on this.

any update @SCR01 ?

@4darsh-Dev yea, I just got little confused but surely do it soon.

⚠[URGENT]⚠ Reminder for Closing of Contribution period for GSSoC 24'
Finish your work and raise the PRs ASAP to get your points.
Thanks💗 for your valuable Contributions!

GSSoC 24 has been completed Finally'
It was a great experience working with you all
Thanks💗 for your valuable contributions!
PA nomination has been started, Do fill out the forms soon. Share your experiences and let's connect on socials