dns/ddclient: no-ip broken

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[X] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[X] I have searched the existing issues, open and closed, and I'm convinced that mine is new.
[X] The title contains the plugin to which this issue belongs

Describe the bug
My no-ip configuration with the new ddclient gives errors, while the dyndns config works fine. It also doesnt show up in the dashboard with the widget and/or service.
To Reproduce
Steps to reproduce the behavior:

have a valid dyndns no-ip configuration
uninstall dyndns plugin
install ddclient plugin
configure ddclient settings with:

Account - same as in the dyndns config
general settings - interval: 300
check ip method: no-ipv4 (dyndns doesnt seem to work for me either, but doesnt give the[meta sequenceId="1"] WARNING: cannot connect to ip1.dynupdate.no-ip.com:80 socket: IO::Socket::INET: Bad hostname 'ip1.dynupdate.no-ip.com'  )

Expected behavior
The same as with dyndns: an update on my no-ip cached ip, or at least log indications that the action is not necessary like my first log-line.
Relevant log files
Logs give me the following (did some reinstalls of the plugin, and change of settings, as well as reboots... but to no avail):
2022-02-03T21:36:57	Notice	ddclient[79495]	 24746 - [meta sequenceId="1"] WARNING:  file /usr/local/etc/ddclient.conf: file /usr/local/etc/ddclient.conf must be accessible only by its owner (fixed).
2022-02-03T21:16:56	Notice	ddclient[79495]	 49526 - [meta sequenceId="1"] WARNING:  file /usr/local/etc/ddclient.conf: file /usr/local/etc/ddclient.conf must be accessible only by its owner (fixed).
2022-02-03T21:01:56	Notice	ddclient[79495]	 89856 - [meta sequenceId="2"] WARNING:  found neither ipv4 nor ipv6 address
2022-02-03T21:01:56	Notice	ddclient[79495]	 86145 - [meta sequenceId="1"] WARNING:  cannot connect to ip1.dynupdate.no-ip.com:80 socket: IO::Socket::INET: Bad hostname 'ip1.dynupdate.no-ip.com'
2022-02-03T20:55:21	Notice	ddclient[36769]	 82331 - [meta sequenceId="1"] WARNING:  file /usr/local/etc/ddclient.conf: file /usr/local/etc/ddclient.conf must be accessible only by its owner (fixed).
2022-02-03T20:50:21	Notice	ddclient[36769]	 48716 - [meta sequenceId="2"] WARNING:  found neither ipv4 nor ipv6 address
2022-02-03T20:50:21	Notice	ddclient[36769]	 44592 - [meta sequenceId="1"] WARNING:  cannot connect to ip1.dynupdate.no-ip.com:80 socket: IO::Socket::INET: Bad hostname 'ip1.dynupdate.no-ip.com'
2022-02-03T20:46:31	Notice	ddclient[70934]	 35353 - [meta sequenceId="1"] WARNING:  file /usr/local/etc/ddclient.conf: file /usr/local/etc/ddclient.conf must be accessible only by its owner (fixed).
2022-02-03T20:37:56	Notice	ddclient[54346]	 19 - [meta sequenceId="1"] WARNING:  file /usr/local/etc/ddclient.conf: file /usr/local/etc/ddclient.conf must be accessible only by its owner (fixed).
2022-02-03T20:32:56	Notice	ddclient[54346]	 57229 - [meta sequenceId="1"] WARNING:  updating ddnsact54858997974632315688744163132698956323.ddns.me: nochg: No update required; unnecessary attempts to change to the current address are considered abusive

Additional context
The things I did today:

install the new 22.1 release from 21.7.
install the plugin ddclient
uninstall the dyndns plugin
configure the ddclient
reinstall the ddclient (should fix the "not showing in dashboard" issue as written in https://github.com/opnsense/plugins/issues/2783#issuecomment-1025585476 , but for me this doesnt solve it)
reboot
change the check ip method
revalidate my account credentials

Environment
OPNsense 22.1 (amd64, OpenSSL)
Intel(R) Core(TM) i3-10100 CPU @ 3.60GHz (4 cores, 8 threads)

could be a dns configuration issue on the machine, you can always check if /usr/local/etc/ddclient.conf contains the correct config, same service fetches an address without issues on my end.

Thank you for your quick response!
After looking back at my settings I finally used my brain and turned on verbose logging for the Dynamic dns under the advanced mode slider menu. Turns out it was working fine all along, but wasn't showing any time updates on my no-ip account because the ip was already set to the current one. The previous dyndns plugin showed me the status on the dashboard widget, but now I only went with the info on no-ip and incorrectly assumed it wasn't working.
So although this was entirely my fault, I think it would be nice to see the widget back in de dashboard like before so we can quickly glance over at the cached ip.
secondly, the service still isn't showing up under the services for me, but it's running fine, so that might just be a GUI thing.
Anyways, this turned out to be a non-issue, so it can be closed safely. Thanks for your time.


support Decimal type

Decimal is not the same as float, provide a Decimal type to let the user handle it explicitly, not already use float instad.

LGTM


Flutter webapp show blank screen on  Safari "Private browsing mode"  caused by "Hive.openBox() "will not be executed.

Steps to Reproduce

Just init a flutter project with Hive database as normally
var box = await Hive.openBox('testBox');
Build flutter webapp in release mode
Open the webapp on the safari "Private browsing mode"
The web will display a blank screen
In the console log, I saw the app stuck on that line var box = await Hive.openBox('testBox');
If we wait for a litter, and reload the Hive box could open, and the app run as normal.
I think in private browsing, the indexed database needs a litter time to create. So the first time the code will not work.

Code sample
var box = await Hive.openBox('testBox');

Version

Platform: iOS, Mac
Flutter version: 2.0.0
Hive version: 2.0.4

Currently, I am making a litter hack like this to work around  with this problem.
First

track to know the flutter could run as expected or not

  <script>
    if ('serviceWorker' in navigator) {
      window.addEventListener('flutter-first-frame', function () {
        navigator.serviceWorker.register('flutter_service_worker.js');
        try {
          var storage = window.sessionStorage;
          var loadKey = "flutter-first-frame";
          storage.setItem(loadKey, "1");
          console.log('flutter-first-frame');
        } catch (ex) { }
      });
    }
  </script>

Second

If the flutter web stay blank after a while , reload the page
Add this script some where in the body.

<script>
  var isSafari = /^((?!chrome|android).)*safari/i.test(navigator.userAgent);
  if (isSafari) {
    setInterval(
      function () {
        var storage = window.sessionStorage;
        try {
          var loadKey = "flutter-first-frame";
          var loadValue = storage.getItem(loadKey);
          if (loadValue == null) {
            storage.setItem(loadKey, "1");
            location.reload(true);
            console.log('work around fix the error : safari-blank-screen problem caused by Hive database open box not executed');
          }
        } catch (e) {
          console.log(e);
        }
      },
      3500
    );
  }
</script>


Reset-GcSqlSslConfig

The cmdlet and tests for Reset-GcSqlSslConfig

I work for google

Your synopsis is a little long, but otherwise LGTM.


Refactoring prototypes (without kafka yet).

Fixes #130
Try out
beamlime --image-path beamlime_plot.png

This command will save the plot into beamlime_plot.png.
What's included
Core

Application components concept, handler, daemon and application.

In the previous prototypes in test helper package, all steps were implemented as daemons.
They are still just concept, and doesn't have a concrete low-level helper modules or a framework yet.
I just tried using those terms for easier communication among us within the project.

Multiple temporary/constant providers setting context manager moved from helper package to constructors module.

Pre-made handlers and daemons

Random event data generator (no log or start-message)
Dummy workflow (Wavelength converting and histogramming, uses some hard-coded values)
Plotting handler that saves a png file.
MessageRouter daemon

It's not a lower-level message routing framework that I tried to implement here. It's just one of pre-made daemons that took some of works that DataReductionApp was doing.

Workflow handler that runs the dummy workflow on the chunk of data.
DataReductionApp as a highest-level interface.

Documentation

Jupyter and command line examples docs/getting-started/live_data_reduction.ipynb

Related next tasks
TODO

[x] Update docstrings
[x] Update documentation and write more explanations  (Maybe briefly, mostly about future implementation from here?)
[x] Update/add tests if needed(?)

Currently looks like this:
Screencast from 02-26-2024 09:35:57 AM.webm

Unless there are essential to-dos, we should get the ready as quickly as possible (do the rest in later PRs), since it will simplify coordination for follow-up tasks. Moving the Kafka bits is not required yet.

Unless there are essential to-dos, we should get this ready as quickly as possible (do the rest in later PRs), since it will simplify coordination for follow-up tasks.

@SimonHeybrock
Yes...! I tried to extract the bare minimum and make it as clear as possible, what each part of code does...

Moving the Kafka bits is not required yet.

Yeah, that's why I didn't include it here.

I tried to refactor it in a more clear way...
classDiagram
    HandlerInterface <|-- DataReductionHandler
    HandlerInterface <|-- PlotSaver

    DaemonInterface <|-- DataStreamSimulator
    DaemonInterface <|-- MessageRouter

    ApplicationInterface <|-- DataReductionApp

    class HandlerInterface {
        <<interface>>
    }
    class DataReductionHandler {
        process_message()
    }
    class PlotSaver {
        save_histogram()
    }
    class DaemonInterface{
        <<interface>>
        async run()
    }
    class DataStreamSimulator
    class MessageRouter {
        async route()
        async send_message_async()
    }
    class ApplicationInterface{
            <<interface>>
            daemons: tuple[DaemonInterface]
            tasks: list[asyncio.Task]
            loop: asyncio.EventLoop
            register_handlers()
            run()
        }
        class DataReductionApp {
            data_stream_listener: DataStreamSimulator
            message_router: MessageRouter
            data_reduction_handler: DataReductionHandler
            plot_handler: PlotSaver
        }

And I also tried to run a benchmark with this implementation and it looks very similar:
online-5-chunk(s)-old is the previous implementation, and the online-5-chunk(s)-new is the new one.
Both of offline-* are also collected again so it was using same pipeline as online-5-chunk(s)-new.

I think the overall architecture makes sense. But I'm confused about ApplicationInterface. Why does it need a base class? Does any part of the code depend on the application?

That is true... what I really wanted is, to allow customizing daemons and handlers.

I think the overall architecture makes sense. But I'm confused about ApplicationInterface. Why does it need a base class? Does any part of the code depend on the application?

That is true... what I really wanted is, to allow customizing daemons and handlers.

In the current implementation, that would require writing a completely new application class, no? So the interface doesn't accomplish much.

I think the overall architecture makes sense. But I'm confused about ApplicationInterface. Why does it need a base class? Does any part of the code depend on the application?

That is true... what I really wanted is, to allow customizing daemons and handlers.

In the current implementation, that would require writing a completely new application class, no? So the interface doesn't accomplish much.

No, not really... you just need to overwrite daemons property or register_handling_methods.
then you can use run method as it is, in the interface class.
I made it this way so that you can just type-hint the class attribute
so that Factory can just inject those properties.
But I had a second thought about this... I think it should have a method that allows to register handling methods and daemons into this application instance...

But I had a second thought about this... I think it should have a method that allows to register handling methods and daemons into this application instance...

Yes, try using composition instead of inheritance.

Now it looks like this...!
classDiagram
    HandlerInterface <|-- DataReductionHandler
    HandlerInterface <|-- PlotSaver

    DaemonInterface <|-- DataStreamSimulator
    DaemonInterface <|-- MessageRouter

    class HandlerInterface {
        <<interface>>
    }
    class DataReductionHandler {
        process_message()
    }
    class PlotSaver {
        save_histogram()
    }
    class DaemonInterface{
        <<interface>>
        async run()
    }
    class DataStreamSimulator
    class MessageRouter {
        async route()
        async send_message_async()
    }
    class Application {
        daemons: list[DaemonInterface]
        tasks: list[asyncio.Task]
        loop: asyncio.EventLoop
        register_handling_method()
        register_daemon()
        run()
    }

And instead of Application registering handlers and daemons itself,
it should be done after building Application instance.
So that Application class doesn't have to know anything about handlers and daemons from the beginning.
So collecting providers is also separated from the Application.

And I also tried to run a benchmark with this implementation and it looks very similar: online-5-chunk(s)-old is the previous implementation, and the online-5-chunk(s)-new is the new one. Both of offline-* are also collected again so it was using same pipeline as online-5-chunk(s)-new.

Is there a reason for using a log-lin scale here instead of log-log? I imagine we expect the time to be linear in the number of events, and in that case a log-log plot illustrates that relationship better.


There option no option to give the User typed text(free text) in the dropdown list

need clarity on this. Could you please explain clearly ?

Yeah,
If user is trying to search for something in the list.. if it is not there in the list.. can i populate tht value in the list dynamically??

redundant requirement. Search functionality should not be adding new items, Just search what is in list. However, upcoming feature is search through service API. which might help you.


Can not open dev environment even build was success

Hi you all, I ran 'npm start dev', everything was ok but I cannot Open http://localhost:3000 in a browser to view the app. I think this issue is urgent, could you pls check it

sorry, I work after I restart my PC


Adding rolloutCiliumPods to true

Description of changes:
rolloutCiliumPods will monitor the cilium-config config map which holds the Cilium configuration and will rollout cilium agent pods when there is a new change.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jaxesn, vignesh-goutham
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jaxesn,vignesh-goutham]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


ROX-17674: Move sensor integration tests to a separate target

Description
This is a followup to https://github.com/stackrox/stackrox/pull/6399. Here the sensor-integration-tests are removed from nongroovy-e2e and a added to the new target gke-sensor-integration-tests. Additionally, ocp-4-*-sensor-integration-tests are provided as optional (the tests are failing there).
The ocp-4-*-sensor-integration-tests are failing, but they are configured to not run automatically.
The 1:1 translation of the existing sensor-integration-tests are available as gke-sensor-integration-tests and those are passing.
Checklist

[x] Investigated and inspected CI test results

N/A

[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

Testing Performed

CI only with the new targets

/test gke-sensor-integration-tests

@fredrb removing the nginx-sa did not help for the OCP tests. Do you have any other ideas?

Do you have any other ideas?

I think I found two problems with OCP.

OCP cannot resolve the image nginx:1.14.2. I just tried changing the image to k8s.gcr.io/pause:3.1 and the deployment gets created.
There are a lot of errors upserting registries in the secrets dispatcher

kubernetes/listener/resources: 2023/06/14 18:02:35.288463 secrets.go:268: Error: Unable to upsert registry "image-registry.openshift-image-registry.svc:5000" into store: unable to check TLS for registry "image-registry.openshift-image-registry.svc:5000": dial tcp: lookup image-registry.openshift-image-registry.svc: no such host

This makes the test timeout before we even get to sync the initial deployments 😨
I've commented these lines in the secrets dispatcher and it works running:
go test -race -count=1 github.com/stackrox/rox/sensor/tests/connection

Obviously this is not the solution, we need to figure out how to solve those upserts, but at least it seems that there isn't anything fundamentally wrong with the tests 🙂

Thanks @lvalerom ! I was able to reproduce the OCP failures locally with the command you provided.
I have disabled the OCP tests in 5ccdbc8 (replaced it with a dummy). There is a follow-up ticket to work on enabling them: https://issues.redhat.com/browse/ROX-17875

@vikin91 can you remove the OCP sensor integration tests from openshift/release also? They are currently set to run_if_changed which will cause 3 OCP clusters to get spun up if anyone modifies that (large) set of files on a PR even though the test won't run. Just removing the run_if_changed would also work. Then they would need to be explicitly requested via a PR comment to run.


Error: Test Folder ... contains a corrupt app.json

I have a repo with multiple apps and when running the pipeline, if fails with below error in the AnalyzeRepo stage. I have compared encoding, have compared any special characters, etc. and can't find any differences between this one and the other app,json files.
Any suggestions on where to look or what this could be? It seems like it's trying to go through the dependencies of the app.json and something seems to not work as the error seems to be only reported in the catch block of the function:
Why would it try to find the entire app.json as a property somewhere?
RunPipeline action failed. Error: Test folder .\MyApp.Test, specified in .AL-Go\settings.json, contains a corrupt app.json file. Error is The property '@id=b0fdf3e6-dbfa-4916-90d3-f75f41c03ce3; name=MyApp.Test; publisher=NAV-X; version=<IP_ADDRESS>; brief=BlahBlahBlah (and blah/blah); description=BlahBlahBlah (and blah/blah); privacyStatement=https://nav-x.com/nav-x-privacy-policy/; EULA=https://nav-x.com/nav-x-license-agreement/; help=https://docs.nav-x.com/en/business-central/; url=http://nav-x.com; logo=./AppLogo.png; contextSensitiveHelpUrl=https://docs.nav-x.com/0/business-central/; dependencies=System.Object[]; screenshots=System.Object[]; platform=<IP_ADDRESS>; application=<IP_ADDRESS>; target=Cloud; idRanges=System.Object[]; resourceExposurePolicy=; runtime=9.0; supportedLocales=System.Object[]; features=System.Object[].Id' cannot be found on this object. Verify that the property exists.. Stacktrace: at , C:\github-01-1_work_actions\microsoft\AL-Go-Actions\v2.0\AL-Go-Helper.ps1: line 652 at , C:\github-01-1_work_actions\microsoft\AL-Go-Actions\v2.0\AL-Go-Helper.ps1: line 612 at , C:\github-01-1_work_actions\microsoft\AL-Go-Actions\v2.0\AL-Go-Helper.ps1: line 584 at AnalyzeRepo, C:\github-01-1_work_actions\microsoft\AL-Go-Actions\v2.0\AL-Go-Helper.ps1: line 565 at , C:\github-01-1_work_actions\microsoft\AL-Go-Actions\v2.0\RunPipeline\RunPipeline.ps1: line 77 at , C:\github-01-1_work_temp\6eabdbc6-08cb-419b-8a22-023300a4dd41.ps1: line 1 at , : line 1

Looks like a bug.
If you can give me access to the repo, I will troubleshoot and fix (and try to give a better error if this is an error in the app.json or like)

Thank you!
Just gave you access. In the past action runs, you should be able to see the issue.

Is it OK if I temporarily change the repo to use a developer version of AL-Go to debug?
We can always change it back again

Yes, no problem.

Got back to this now - seems like I don't have permissions to run workflows:-(

When I invited you, I gave you the admin role. It still says that your invitation is pending.
It seems I invited you to the org and didn't wait with adding you to the repo. I gave you admin access on the repo.

After using my dev version of AL-Go - it fails with another error.
That error indicates that the apps in the repo cannot co-exist (overlapping object IDs)
It looks like you would have to create a multi-project repo, which looks like this;
project1\

.AL-Go\
app1\
app2
project2\
.AL-Go\
app3\
app4\

Basically a folder per project - each with a .AL-Go folder and the apps that are inter-dependant.
Hope that makes sense.

Thank you. It does make sense and is definitely a better error message lol.
So, does this mean I cannot define the same ID range  in multiple apps? As they're PTEs, having the same range just made it easier, because object uniqueness is handled differently.Or does it actually check the IDs of all objects in the repo?

Yes you can, but then the PTEs needs to be in different projects or different repositories.
All apps in one project will be built, published and tested together.

So, I went through all apps to make sure that the app.json has different ID ranges for each of the apps.
Also reviewed the apps and can't find any that have the same object IDs, field IDs, or ID ranges.
However, I am still getting the error.
I ran update system files again to see, if that would do anything. Nope
Completed the pull request and ran CI/CD in main only. Nope
I can't understand it, especially, since I have multiple repos that are the same way (multiple apps, multiple PTEs) and they work without a problem.

Found my error.
Have hotfixed v2.0 (and preview) - now it displays the correct error, so that you can fix yours.-)
You have two test apps using the same appid.

ugh, how did that happen... oh well, thank you very much!


Lunr search index is not working

the search on Franklinjl.org doesn't work.

Now fixed, the javascript had been commented and there was a clash with the focus.


[RESTEASY-1728] Look for resteasy.allowGzip system property in Regist…

…erBuiltin
This is a proposal for a "fix" applying to both client and server side. Basically every time a new ResteasyProviderFactory instance is created,we would go and check the system property.
This said, we can probably do better... please provide feedback.

This is related to the issue that got me thinking about a general purpose facility for checking system properties. Something like
         ResteasyConfiguration context = ResteasyProviderFactory.getContextData(ResteasyConfiguration.class);
         if (context != null)
         {
            String s = context.getParameter("resteasy.allowGzip");
            if (s == null)
            {
                 s = AccessController.doPrivileged(new PrivilegedAction<String>() {
                        @Override
                        public String run() {
                            return System.getProperty("resteasy.allowGzip");
                        }
            }
            if (s != null)
            {
               allowGzip = Boolean.parseBoolean(s);
            }
         }

@ronsigal Take a look at Elytron's security manager facility, https://github.com/wildfly-security/wildfly-elytron/blob/master/src/main/java/org/wildfly/security/manager/WildFlySecurityManager.java#L1114.

@ronsigal , sure, it's related, and this is probably yet another confirm that we should be implementing your proposal (in that case we'd refactor this fix a bit, should be possible in backward compatible way, right?).
@iweiss , what do you actually want to suggest with the link above?

Do you guys think this can be used in 2.3.x branch as it is, so that it's not holding EAP release any longer? If there's going to be some refactoring later on, this branch could perhaps live without it, if it doesn't change a substance of the fix.


Errors during build

Cannot build because of the following error.
Could not determine the dependencies of task ':app:processDebugResources'.

Could not resolve all task dependencies for configuration ':app:debugRuntimeClasspath'.
Could not find com.github.LuckSiege.PictureSelector:picture_library:v2.5.9.
Searched in the following locations:

https://dl.google.com/dl/android/maven2/com/github/LuckSiege/PictureSelector/picture_library/v2.5.9/picture_library-v2.5.9.pom
- https://dl.google.com/dl/android/maven2/com/github/LuckSiege/PictureSelector/picture_library/v2.5.9/picture_library-v2.5.9.jar
- https://jcenter.bintray.com/com/github/LuckSiege/PictureSelector/picture_library/v2.5.9/picture_library-v2.5.9.pom
- https://jcenter.bintray.com/com/github/LuckSiege/PictureSelector/picture_library/v2.5.9/picture_library-v2.5.9.jar
- https://developer.huawei.com/repo/com/github/LuckSiege/PictureSelector/picture_library/v2.5.9/picture_library-v2.5.9.pom
- https://developer.huawei.com/repo/com/github/LuckSiege/PictureSelector/picture_library/v2.5.9/picture_library-v2.5.9.jar
- https://storage.googleapis.com/download.flutter.io/com/github/LuckSiege/PictureSelector/picture_library/v2.5.9/picture_library-v2.5.9.pom
- https://storage.googleapis.com/download.flutter.io/com/github/LuckSiege/PictureSelector/picture_library/v2.5.9/picture_library-v2.5.9.jar
Required by:
project :app > project :images_picker

@jabirmayar  hi,your problem look likes error with download url, you can add maven { url 'https://jitpack.io' } in your build.gradle to test.

Sure. I will try and report.


dotnet not installed error

I have dotnet installed in windows 11 in C:/Program files and also in C:/Program files(x86) folder but, this application giving me "dotnet not installed , do you want to download now? - yes / no" error to run.
and I have dotnet version 8.0.202 and 8.0.206 installed in my pc.

Ensure that the .NET SDK and runtime are correctly installed. You can verify this using the command line:
Open Command Prompt or PowerShell.
dotnet --list-sdks
dotnet --list-runtimes
Update PATH Environment Variable
Open Environment Variables:

Right-click on This PC or Computer on the desktop or in File Explorer.
Select Properties.
Click on Advanced system settings.
In the System Properties window, click on the Environment Variables button.

In the System variables section, find the Path variable and click Edit.
Ensure that C:\Program Files\dotnet and C:\Program Files (x86)\dotnet are listed.


Gear Checks reports missing enchant on Thrown Items

Gear Checks reports that ranged slot is missing an enchant if you're using a thrown item, but as far as I know there doesn't exist any enchants at all for thrown items. Could be the same for relics/wands but I don't have characters to test that.
CSP Improved version: 1.2.7

Thanks I'll fix that.
I started out with exception for all off-hands (except shield / weapon) but folks informed me there's enchants for frills.
It already excludes wands.
I should probably exclude relics from enchant checks and I'll add thrown.

I checked and I'm actually already excluding armor subtypes in ranged slot (so relics shouldn't need a change).
It's only thrown that need added to exclusions.

fixed with 60367cde02746f731839307a39dbb38908c9182c


Update mssp-list.md

Updated BlueVoyant Description as per their official website

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/9263

@JohanFreelancer9 Please copy review

@yogkumgit macapara is the Author

Docs Build status updates of commit 8a7f8f5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/defender-endpoint/mssp-list.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 0634429:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/defender-endpoint/mssp-list.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 0634429:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/defender-endpoint/mssp-list.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


homepage section and subsection featured articles

@peterp please see screenshots

@Mitso I've left some comments in the PR.

@peterp please review


Fixed pandas dimension ingestion

The changes I made to how the pandas interface ingests dimensions weren't quite correct so this restores the old behavior while also fixing an issue with too many dimensions being extracted as vdims.
This time I also added a bunch of tests to ensure it's actually working properly.

Push build passed, merging.


Fixing CLS for communities

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #92
Test URLs:

Before: https://main--hubblehomes-com--aemsites.hlx.page/new-homes/idaho/boise-metro/nampa/adams-ridge
After: https://issue-92--hubblehomes-com--aemsites.hlx.page/new-homes/idaho/boise-metro/nampa/adams-ridge

Couple things that jump out visually:

In the original site, the selected tab within the subnavigation component is blue, whereas it appears grey in the Pull Request.
The list styling within the overview block is lacking the blue color for the list items.
The button height in the subnavigation is larger in the original site compared to the Pull Request.


There is no code

same with cla signed - was there any info I read over that it follows later? confused

Microsoft will open source Coco Framework in 2018.


[Feature] Revisit Ray logging doc

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description

P0

TODO: Delete the sections from "Set up logging on a DaemonSet with Fluent Bit" to "Check the CloudWatch dashboard".

The section that installs FluentBit via DaemonSet only persists logs from STDOUT/STDERR. However, most Ray logs are located in /tmp/ray/session_latest. See this comment for more details.

TODO: sidecar solution

The FluentBit sidecar container redirects logs from /tmp/ray/session_latest/logs to Grafana Loki.

TODO: Allow users to associate logs from different RayCluster / RayJob CRs

P1

TODO: Use Configuration API (code) to set up the logging container instead of configuring each custom resource.
Consider removing the section for RAY_LOG_TO_STDERR=1. (https://github.com/ray-project/kuberay/issues/2515#issuecomment-2476761824)

Use case
No response
Related issues
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

@kevin85421 could you please elaborate on the idea behind removing RAY_LOG_TO_STDERR from the documentation? Does this demonstrate a longer-term shift in support for logging to stderr / stdout?
My team uses this flag extensively as we use an external Grafana stack to scrape stdout / stderr workload logs. We intentionally don't want to write to log files on disk and we do not need dashboard / driver log redirection in production.

My team uses this flag extensively as we use an external Grafana stack to scrape stdout / stderr workload logs. We intentionally don't want to write to log files on disk and we do not need dashboard / driver log redirection in production.

Good to know. In that case, maybe we should leave the section there after all. I considered removing it from the doc because it will cause some dashboard features don't work.

@win5923 will take this issue.


DAP: introduce Rdbg Trace Inspector

We'll introduce a new feature Rdbg Trace Inspector. This is useful when tracing in VS Code.
https://user-images.githubusercontent.com/59436572/223106427-2b808adb-b38e-4c44-9050-dbb8cd0cc893.mov
How to use it

Click on "Start Trace" in TRACE view
Execute the program

That's it! You can check the location and returned value by clicking the trace log.

I've been wanting this feature for a while, this looks super cool 🎉
How can we select the even type with this UI (e.g. exclude the line events)? From the video I don't see an option to do so.
Also, is it possible to have some protocol tests for it?

how can we select the even type with this UI (e.g. exclude the line events)?

We can select the option as follows:
https://user-images.githubusercontent.com/59436572/223157318-70889b66-72fd-4bb1-b206-e20e415222a0.mov

Also, is it possible to have some protocol tests for it?

Yeah, I think that we can add tests.

Yeah, I think that we can add tests.

@ono-max Thanks. Could we first have some tests in the same PR?

I'll try it, but I'm not sure if I can add tests in the same PR because I'm busy. Sorry.


upgrade RN in example to 0.64.2

I think this would break expo, no https://blog.expo.dev/expo-sdk-42-579aee2348b6? @cruzach

Yeah, it would break running the example in Expo Go (although we do still have the snack example, but not ideal)
FYI we're looking into upgrading to RN 64 in the next SDK (43)


extension doesn't work in fileopen dialog window

My extension working well in explorer, but in the file-open dialog, neither the icon handler nor context menu handler works. I checked the log, looks like the extension is not even loaded.

Hi @kkguo is this for any file open dialog? Or just one from a certain program? They can be created with various options, some of which may possibly interfere with the shell extensions...

No matter from what program, I cannot see shell extension in file open dialog window, I can see some extension, but I guess they are native win32. But I recently realized it maybe only happen in my machine,  my user didn't report issue like this. I'm using win7 64bit enterprise. I'm trying to dig more, I will close if I can find a clue, or I cannot..

I'm closing this to focus on recent issues, please re-open if needed!


Discover images

not clickable

needs heighliner update https://github.com/NewSpring/Heighliner/issues/83

fixed by https://github.com/NewSpring/Heighliner/issues/83


Fix _call not implemented error in call_and_ladj

Some transformations do not implement _call, even some of the Pytorch ones like ComposeTranform.
https://github.com/pytorch/pytorch/blob/4869929f32c176dc3e5ea4cd4164b3fd73a0c9ea/torch/distributions/transforms.py#L270-L385
This results in call_and_ladj raising a NotImplementedError.
>>> import zuko
>>> t = zuko.transforms.ComposeTransform([])
>>> t.call_and_ladj(None)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File ".../zuko/transforms.py", line 41, in _call_and_ladj
    y = self._call(x)
  File ".../torch/distributions/transforms.py", line 176, in _call
    raise NotImplementedError
NotImplementedError

Curiously, the test fails if I add ComposeTransform([IdentityTransform(), SinTransform()]). I think this is unrealted, but I think should be investigated?

Hello @felixdivo,
Thanks for reporting the bug and finding a solution. Could you address my comments before I merge?

the test fails if I add ComposeTransform([IdentityTransform(), SinTransform()])

This is because SinTranform is bijective between $-\frac{\pi}{2}$ and $\frac{\pi}{2}$. In ComposeTransform, the domain constraint is invisible for the test. So it defaults to $[-5, 5]$.
As a side note, per the contribution guidelines:

If you found a bug and want to fix it, please create an issue reporting the bug before creating a pull request. Similarly, if you want to add a new feature, first create a feature request issue. This allows to separate the discussions related to the bug/feature, from the discussions related to the fix/implementation.

As a side note, per the contribution guidelines

Thanks for the hint and sorry for not reading it. I'll follow it from now on. I wasn't aware of it.


"supportsStartDebuggingRequest" is unset

(VSCode 1.75.0)
Per DAP specification and release notes for 1.73, the adapters should only be using the "startDebugging" reverse request if the client advertises that it's capable of handling it by setting "supportsStartDebuggingRequest" to true during initialization. However, VSCode does not advertise this capability at all despite supporting the feature:
             {
                 "seq": 1,
                 "type": "request",
                 "command": "initialize",
                 "arguments": {
                     "clientID": "vscode",
                     "clientName": "Visual Studio Code",
                     "adapterID": "python",
                     "pathFormat": "path",
                     "linesStartAt1": true,
                     "columnsStartAt1": true,
                     "supportsVariableType": true,
                     "supportsVariablePaging": true,
                     "supportsRunInTerminalRequest": true,
                     "locale": "en",
                     "supportsProgressReporting": true,
                     "supportsInvalidatedEvent": true,
                     "supportsMemoryReferences": true,
                     "supportsArgsCanBeInterpretedByShell": true,
                     "supportsMemoryEvent": true
                 }
             }

Since conforming adapters have to assume the default to be false in practice for backwards compatibility, it makes the reverse request unusable in practice.

Another oversight, thanks


Project panel - clicking on class while exploring SWF results in an exception

When I'm browsing classes or symbols inside a compiled SWF in the project panel, if I click on any class or symbol in the tree, I get this exception:
The given path's format is not supported.

   at System.Security.Util.StringExpressionSet.CanonicalizePath(String path, Boolean needFullPath)
   at System.Security.Util.StringExpressionSet.CreateListFromExpressions(String[] str, Boolean needFullPath)
   at System.Security.Permissions.FileIOPermission.AddPathList(FileIOPermissionAccess access, AccessControlActions control, String[] pathListOrig, Boolean checkForDuplicates, Boolean needFullPath, Boolean copyPathList)
   at System.Security.Permissions.FileIOPermission..ctor(FileIOPermissionAccess access, String[] pathList, Boolean checkForDuplicates, Boolean needFullPath)
   at System.IO.Path.GetFullPath(String path)
   at CodeRefactor.PluginMain.OnTreeSelectionChanged()
   at CodeRefactor.PluginMain.HandleEvent(Object sender, NotifyEvent e, HandlingPriority priority)
   at PluginCore.Managers.EventManager.DispatchEvent(Object sender, NotifyEvent e)

it will be fixed it today, sorry

Thanks!


can you support old VS version?

my code is in the old VS.So I can't use this tool!

What version of Visual Studio are you using? And what scenario do you want to use MIEngine for?

VS2012 !!! My project is built by VS2012~ there are many procject in my company!

Visual Studio installs side-by-side (meaning you can have multiple version of Visual Studio installed at the same time) so if your company qualifies, you can install VS 2015 Community Edition for free.
If your company is too large to qualify, you could probably build your own version of this package and install the .vsix into VS2012 with little effort.


Added the cross origin CORS to config

Added CORS (Cross-origin resource sharing ) in config.

I have added a test case grpc cors, closing this request


Inconsistency on updating tags

In my case, I'm using the library along with Redis, and it seems I've found a bug related to how the tags are formed.
I was attempting to invalidate all messages with the same tag, but it only deleted the last one
Steps for Reproduction

Set up a project using redis and with the marshaler cache;
Save any kind of message, with a tag, lets say "myTag" with key "abc";
On redis, there will be a key called "gocache_tag_myTag", with value "abc";
Save another message, with the tag "myTag" and key "def";
The key "gocache_tag_myTag" now has the value "def".

With this, if I attempt to invalidate messages with the tag "myTag", only the message with key "def" will be cleaned up.
Expected behavior:

Set up a project using redis and with the marshaler cache;
Save any kind of message, with a tag, lets say "myTag" with key "abc";
On redis, there will be a key called "gocache_tag_myTag", with value "abc";
Save another message, with the tag "myTag" and key "def";
The key "gocache_tag_myTag" should have the value "abc,def"

Versions:
Which versions are you running?
v1.0.0

It seems the problem is already solved on the master branch. When can you guys tag a new version so it works nicely with Go mod?

The tag has been created


syntax error in DKIM record with BIND 9.16.1-Ubuntu (Stable Release)

Hi,
I've copied the dkim TXT record to my DNS and I am receiving a syntax error:
listsdkim._domainkey.lists.sa.[tld] IN TXT "v=DKIM1; k=rsa; p=MIIBIjANBgkqhkiG9w0BAQEFAAOCAQ8AMIIBCgKCAQEA75p6i6YYw51PuRd9eSGgVwC09+d8PNSEX6m2qxb3GF0hlUzjFkR2uab0j9fLigNSI4cfqR6XDL5wFqf4VOuiOqvRsdgQgYf+3RFLZgKy+YwcKNHPYrKcwnXQZSKaJylJ9X9DlqJFAVr4m6XdOowCuVAtIJdsAh2eJ9HbkUGuqVVl+kobR7HzFGt7Yo6bdaQaQu8v70mejGg7YYaf3jaGg1knVJS2rz5ay3Di3xBjxRewRic0L3PO5c5onyCtOLSkrCuhg7oV+FaIqpQ50EG/BYJJHNChn0e1DP5T34o2fK/audQyCtaNzEujSO2nvTrfu2k//2app2QvgRZmnEj6CwIDAQAB"
[tld] = I removed my domain name
The error in bind is:
dns_rdata_fromtext: /etc/bind/primary/sa/sa:275: syntax error
I can generate a separate dkim which does work:
lists._domainkey.sa.[tld]     IN      TXT             "v=DKIM1;t=s;p=MIGfMA0GCSqGSIb3DQEBAQUAA4GNADCBiQKBgQDHaugpuWW72XLAGF+jXTALY89jpYUYt2Ca7j9tH2DXU7SDp8T1XcU3Ol5oLX5w8wGwTs/G361Rr4/W98c37LFoZWkte21JO9TGB944jqIkL1FXbJF4GsaDmetcaGO63ICAzBbNIx/8cdwlvbVTdFWsvWgf4DCyIVvhwr+8qc42hwIDAQAB"
Any ideas? Can I just use my key ?  Can i update the private key somewhere on the image ?
Thanks for any help you can provide.

I think I can just copy my private and public key in to the relevant files in /etc/exim4/tls.d to use the record I generated.
I'm leaving this issue open in case you want to comment on why the record the container is generating fails.

I believe the key generated is too large for the bind server but I can confirm that by generating my DKIM and copying the private and public keys in to the appropriate file in /etc/exim4/tls.d (which is mapped to the directory a launch the mailman docker container), that when I regenerate the container it generates a DNS TXT record which is able to be used in bind.

Hi,
The container self-generates a 2048 bits DKIM key which is becoming the new standard that should be accepted in most cases.
You can of course use your own  DKIM key as you said by replacing the relevant files in /etc/exim4/tls.d/
Regarding the error you reported, it seems to be a syntax error that might be due to a miss-configuration of the container EMAIL_HOST environment variable since the proposed DKIM TXT record by the container is built this way:
listsdkim._domainkey.${EMAIL_HOST} IN TXT \"v=DKIM1; k=rsa; p=$key
And you reported that the container suggested a TXT record starting with
llistsdkim._domainkey.lists.sa.[tld] IN TXT ...
Which means that the EMAIL_HOST environment variable was set to lists.sa.[tld]
but the DKIM TXT record you reported working starts with:
lists._domainkey.sa.[tld] IN TXT ...
Which means that your email host MX RECORD was actually set in your DNS records to be sa.[tld] instead of the lists.sa.[tld] declared in ${EMAIL_HOST} environment variable of the container.
I think that if you make match the MX RECORD of your DNS with the ${EMAIL_HOST} environment variable of the container it might fix the observed error and you might benefit from using a 2048 bits DKIM key.


MSBuild 17.8 static property functions for System.OperatingSystem

A future version of MSBuild will support static property functions for System.OperatingSystem.  That feature should be documented here, when it is known which version of MSBuild will ship with this change.  The feature was requested in https://github.com/dotnet/msbuild/issues/5982 and implemented in https://github.com/dotnet/msbuild/pull/8935.
These property functions will return information about the operating system on which MSBuild is running.  For example, if your project targets Linux and you build it on macOS, the property functions will return information about macOS.
In MSBuild running on .NET (e.g. dotnet build), all static methods of the System.OperatingSystem class will be callable as static property functions.
In MSBuild running on .NET Framework (e.g. MSBuild.exe), only the following methods of System.OperatingSystem will be callable as static property functions.  MSBuild will implement them internally, because System.OperatingSystem does not define them on .NET Framework.  Other methods such as System.OperatingSystem::IsTvOS will not be callable, but developers won't need them because .NET SDK cannot be installed on those operating systems.

System.OperatingSystem::IsOSPlatform
System.OperatingSystem::IsOSPlatformVersionAtLeast
System.OperatingSystem::IsLinux
System.OperatingSystem::IsFreeBSD
System.OperatingSystem::IsFreeBSDVersionAtLeast
System.OperatingSystem::IsMacOS
System.OperatingSystem::IsMacOSVersionAtLeast
System.OperatingSystem::IsWindows
System.OperatingSystem::IsWindowsVersionAtLeast

The Microsoft.Build.Framework.OperatingSystem type is an implementation detail and should not be documented.

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

ID: 82a1a083-bea0-e53e-dc9e-78f201ac1314
Version Independent ID: 904dae68-65cb-ec55-2fb3-1075ceaf6f1c
Content: Property Functions - MSBuild
Content Source: docs/msbuild/property-functions.md
Product: visual-studio-windows
Technology: msbuild
GitHub Login: @ghogen
Microsoft Alias: ghogen

@KalleOlaviNiemitalo -- Kalle, thank you for your very detailed feedback.
@ghogen -- Gordon, please look into this issue.

Docs added, under review internally.


irmin-pack: introduce shallow GC API changes, V0 GC compatibility

This is a partial view of the current WIP (which supersedes https://github.com/mirage/irmin/pull/2168)
In particular the testing code for the V0 compatibility is not pushed yet.

@Firobe are you looking for feedback?

Sure, feel free to roast what's already pushed :)

Codecov Report

Merging #2190 (49a0c84) into main (b080cc7) will decrease coverage by 0.06%.
The diff coverage is 23.25%.

:exclamation: Current head 49a0c84 differs from pull request most recent head 3fe9d00. Consider uploading reports for the commit 3fe9d00 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main    #2190      +/-   ##
==========================================
- Coverage   68.12%   68.07%   -0.06%
==========================================
  Files         135      135
  Lines       16255    16274      +19
==========================================
+ Hits        11074    11078       +4
- Misses       5181     5196      +15

Impacted Files
Coverage Δ

src/irmin-pack/unix/gc.ml
74.76% <0.00%> (-2.91%)
:arrow_down:

src/irmin-pack/unix/gc_worker.ml
4.61% <0.00%> (+0.20%)
:arrow_up:

src/irmin-pack/unix/inode.ml
10.00% <0.00%> (-2.50%)
:arrow_down:

src/irmin-pack/unix/pack_key.ml
65.38% <0.00%> (-5.45%)
:arrow_down:

src/irmin-pack/unix/pack_store.ml
78.60% <9.09%> (-4.12%)
:arrow_down:

src/irmin-pack/unix/store.ml
63.38% <62.50%> (ø)

src/irmin-pack/unix/file_manager.ml
83.75% <66.66%> (-0.38%)
:arrow_down:

src/irmin-test/store.ml
94.95% <0.00%> (+0.05%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Following our discussion with @metanivek, removed the various flags in the API and instead now we choose the behavior of GC finalisation depending on the presence of a lower layer. Also added a behavior function for the user to check that.


Ability to stream torrent file from the library

With preferential downloading of the portion of the torrent that is currently is being streamed


Ability to disable presets.

Request is pretty simple - ability to disable all the preset stuff. Some people have different approach to managing heating and those presets are just confussing non-technical users.
It would be nice to also disable the modes.
All of those in a sense of NOT exposing this to HA so those are not automatically harvested by HASS.

Yeah, maybe don't close the issue if you've not confirmed with the reporter that this actually work.
You can't set value of 0 if you set minimum allowed temp higher than 0. Sorry, but one setting of temp ranges covering UI and backend configuration is "suboptimal".

Sorry to close the issue to rapidly, before 6.3.0 it was possible to specify 0 at temperature which deactivate the preset.
It is now not possible, that is right. So I have no way now to disable a preset.
I will think about this.

I would suggest a temp setting at the central place - ie "central config" and a secondary min / max setting in thermostat place. That way you can allow global configuration, and then allow people to set per thermostat settings.
Or simply:

allow whatever temp user wants for presets
allow disabling presets via configuration (not through the device or other funky kung-fu)


perf: update textwrap to version 0.7.0

This version of textwrap uses a new wrapping algorithm that allocates
fewer strings on the heap.
The 05_ripgrep benchmarks shows a performance improvement of 3-15%.
The build_help_long benchmark benefits the most since it uses longer
help texts.

This change is

Coverage decreased (-0.4%) to 91.532% when pulling c2d4e63756a6f070e38c16dff846e9b0a53d6f93 on mgeisler:textwrap-0.7.0 into 76869a0d68193093a70dcb63d248a7da91294a16 on kbknapp:master.

Excellent, thanks!


contact information of the logo's maker

Hello.
Currently Pyenv doesn't have a logo. I like rbenv's one a lot. I am ready to pay the artist to do a similar one for pyenv. Can you point me the author?
Thank you.

Hi, I think the "artist" was me, but it was so long ago that I might have invented that fact in my brain. I'm asking someone else to verify.
In any case, there wasn't a professional artist I could refer you to, sorry!

Something like this?

I can't charge for this; it took me 5 minutes to make in Inkscape. You can freely use the PNG, or you can modify the SVG source to your liking.

Hope this helped!

Wow, it's marvellous. PyEnv dudes loved the idea and I am sure they will love this marvel of a logo.  I fill find a way to funnel you a few bucks because you are too awesome not to ;).
Thank you so, so, so much 🙏


Versioning of python packages

Please do not forget to update the Python API version when you change it in configure/CMakeLists

I really think that having two different version numbers for C/Fortran and Python is extremely confusing. If we tell users to get version 2.3.2 of the library, they will automatically do pip install trexio-2.3.2 and will get it wrong. It adds an extra layer of complexity.
As the Python package is distributed with the C library in a single repo, I think we should really have a common version number.

I am sure that most of the time the users do pip install trexio without pinning the version :-)
The motivation to have them different was to allow patches in the Python API which do not affect C/Fortran. For example, recent bug with the loss of precision when reading float numbers was fixed purely on the Python side and this was reflected in the bump of Python patch version, without touching the C library. For me personally it would be more confusing to see a change in a C library version only because of the Python API.
You can sync both C and Python API versions if you want.


Textures for item multiples need to be implemented

i.e. IV_AspirinMultiple.png

Implemented StackCount for Aspirin in cff5a97

Thanks for the reference.  Will apply this to future texture variants!  Just saw this so the current batch of textures just uploaded has not been "hooked up" yet.


pandas version too restrictive

Your pandas requirements are too restrictive,
making conflict with other big packages.
Can you make compatible with pandas > 1.0

This package is compatible with pandas>=0.25.3 with the exclusion of a few patch releases (1.0.{0,1,2} and 1.1.0) that contained serious bugs that affected effective working. All latest patches are fully working (e.g. 1.0.5, 1.1.5,1.2.3). In case you really need one of the bugged pandas versions, feel free to submit a PR patch.


"Error: Authorization page could not be loaded"

Hi, I have 2 user profiles for Youtube under the same google account. I want to import the subscription for the secondary account that gives me the error above. The first user profile seems to work but it shows me my (one) subscriptions multiple times which is weird.
Unfortunately I cannot provide any additional informations. The console and the network informations do not show any errors.

I think I'd need more information about your secondary account, is your account a Google+ account or a YouTube account?

It is a page. This is the Authentification page.

The message appears after I click "allow".

Alright I think I might know what your issue is, the account you are attempting to login with is not a valid Youtube account. As you can see in the following screenshots, I am attempting to login with the account with the red outline but it does not exist in the second screenshot which tells me which YouTube accounts I have on this email.

Well, I can post comments, subscribe to people and even some people follow me so for Youtube it is a Youtube account. I created that account when Youtube integrated Google Plus into Youtube and I never had issues with it. I just don't want to maintain two lists for this. I just want to get notified when somebody uploads a video.

Okay, I'll need you to:

Goto chrome://extensions
Check Developer Mode on the top right
Find YouTube Notifications
Find Inspect views:  and click on background.html
Goto the Network tab and make sure XHR is selected
Attempt to login again with the inspector window open
Once the error pops up again, scroll down to the bottom and click on the row that is RED.
Click on the Response tab and send the entire output

Nope, I have nothing like this when I follow the steps. But that is actually what I did before because I'm used to write my own addons and it does not show me any request/errors.

Sorry for the late reply, it should definitely show the request there because that's the only way you would receive the error otherwise there would be no popup stating there's an error. Also check the settings popup too just in case.

FIXED: If youtube is blocked at your network, you will get error Authorization page could not be loaded.


Fix constraint bounds issue

Fix for #81

Codecov Report

Merging #84 into devel will decrease coverage by 0.01%.
The diff coverage is 85.71%.

@@            Coverage Diff            @@
##           devel      #84      +/-   ##
=========================================
- Coverage   88.7%   88.68%   -0.02%
=========================================
  Files          8        8
  Lines       2381     2386       +5
  Branches     542      543       +1
=========================================
+ Hits        2112     2116       +4
  Misses       167      167
- Partials     102      103       +1

Impacted Files
Coverage Δ

optlang/interface.py
90.04% <85.71%> (-0.08%)
:arrow_down:

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 53730c7...8249842. Read the comment docs.


Allow variable substitution in command line defines

Attempt to fix edge cases where other extensions need to know about the
actual source path.
Resolves: #4

@floriantschopp Please check if exhale works if you override the containmentFolder variable with -D 'containmentFolder=${sourcedir}/api'.

@Holzhaus Thank you for taking care of this. I just tried it with
sphinx-multiversion docs build/html -D 'containmentFolder=${sourcedir}/api'

with the following output
unknown config value 'containmentFolder' in override, ignoring
Failed load config for refs/heads/develop from /tmp/tmpgsg2thm0/0b7a48b9813033e54a78ae157ba9ed43629f084f/docs
unknown config value 'containmentFolder' in override, ignoring
unknown config value 'containmentFolder' in override, ignoring
Failed load config for refs/tags/initial_release from /tmp/tmpgsg2thm0/63ea0c8acb669904a711817b3cb74e314ecffec7/docs
Failed load config for refs/heads/master from /tmp/tmpgsg2thm0/618a0e0154d0c6e6cd84b5ac827140b5a9b84cb0/docs
Running Sphinx v2.4.4
/home/spadmin/.local/lib/python3.6/site-packages/sphinx/util/docutils.py:287: RemovedInSphinx30Warning: function based directive support is now deprecated. Use class based directive instead.
  RemovedInSphinx30Warning)
WARNING: unknown config value 'containmentFolder' in override, ignoring
WARNING: The config value `html_sidebars' has type `list', defaults to `dict'.

Configuration error:
The given `containmentFolder` [/home/spadmin/tmp/maplab_ws/src/maplab_experimental/maplab/docs/api] must be a *SUBDIRECTORY* of [/tmp/tmpgsg2thm0/ee893da17b9dc123dc1b4faa9f5b23a9ad3fc539/docs].

FYI: My conf.py is the following:
import os

name = 'maplab'

on_rtd = os.environ.get('READTHEDOCS', None) == 'True'

if not on_rtd:  # only import and set the theme if we're building docs locally
    import sphinx_rtd_theme
    html_theme = 'sphinx_rtd_theme'
    html_theme_path = [sphinx_rtd_theme.get_html_theme_path()]

html_logo = "logo.png"

#source_parsers = {
#    '.md': 'recommonmark.parser.CommonMarkParser',
#}

extensions = [
    'breathe', 'exhale', 'sphinx.ext.autosectionlabel', 'recommonmark', 'sphinx_multiversion',
]

project = name
master_doc = 'index'

html_theme_options = {
    'canonical_url': 'maplab.asl.ethz.ch',
    'analytics_id': 'UA-XXXXXXX-1',  #  Provided by Google in your dashboard
    'logo_only': True,
    'display_version': True,
    'prev_next_buttons_location': 'bottom',
    'style_external_links': True,
    'vcs_pageview_mode': 'edit',
    'style_nav_header_background': 'black',
    # Toc options
    # 'collapse_navigation': True,
    # 'sticky_navigation': True,
    # 'navigation_depth': 4,
    # 'includehidden': True,
    # 'titles_only': False
}

html_context = {
    'display_github': True,
    'github_repo': 'maplab',
    'github_user': 'ethz-asl',
    'github_version': 'develop-docs',
    'conf_py_path': '/docs/',
}

templates_path = [
    "templates",
]

html_sidebars = [
    "versions.html",
]

# Setup the breathe extension
breathe_projects = {"project": "./doxyoutput/xml"}
breathe_default_project = "project"

# Setup the exhale extension
exhale_args = {
    "verboseBuild": False,
    "containmentFolder": "./api",
    "rootFileName": "library_root.rst",
    "rootFileTitle": "Library API",
    "doxygenStripFromPath": "..",
    "createTreeView": True,
    "exhaleExecutesDoxygen": True,
    "exhaleUseDoxyfile": True,
    "pageLevelConfigMeta": ":github_url: https://github.com/ethz-asl/" + name
}
source_suffix = ['.rst', '.md']

# Tell sphinx what the primary language being documented is.
primary_domain = 'cpp'

# Tell sphinx what the pygments highlight language should be.
highlight_language = 'cpp'

ah ok, the containment folder is inside the exhale_args dict. I have no idea if sphinx allows overriding value inside a dict, but you could try to use -D exhale_args.containmentFolder=${sourcedir}/api instead.

The override seems to work, but the variable seems to be unset:
$ sphinx-multiversion docs build/html -D exhale_args.containmentFolder=${sourcedir}/api
Failed load config for refs/heads/develop from /tmp/tmpk87gkyct/0b7a48b9813033e54a78ae157ba9ed43629f084f/docs
Failed load config for refs/tags/initial_release from /tmp/tmpk87gkyct/63ea0c8acb669904a711817b3cb74e314ecffec7/docs
Failed load config for refs/heads/master from /tmp/tmpk87gkyct/618a0e0154d0c6e6cd84b5ac827140b5a9b84cb0/docs
Running Sphinx v2.4.4
/home/spadmin/.local/lib/python3.6/site-packages/sphinx/util/docutils.py:287: RemovedInSphinx30Warning: function based directive support is now deprecated. Use class based directive instead.
  RemovedInSphinx30Warning)
WARNING: The config value `html_sidebars' has type `list', defaults to `dict'.

Configuration error:
The given `containmentFolder` [/api] must be a *SUBDIRECTORY* of [/tmp/tmpk87gkyct/ee893da17b9dc123dc1b4faa9f5b23a9ad3fc539/docs].

@floriantschopp Thanks for testing. I added some patches. Can you retest, please? If it doesn't work, add --dump-metadata and check the value of the sourcedir key. Is it empty, too?

The override seems to work, but the variable seems to be unset:
$ sphinx-multiversion docs build/html -D exhale_args.containmentFolder=${sourcedir}/api
...

You need to use single quotes around the override! Otherwise your shell will think it's an environment variable and try to replace it before it's passed to sphinx-multiversion. When there is no environment variable sourcedir, it will just replace it with an empty string.
Use it like this:
$ sphinx-multiversion docs build/html -D 'exhale_args.containmentFolder=${sourcedir}/api'

Thank you very much for the fixes and explanations! Seems to work now :-)
Do you see an easy method to define (e.g. in a file in the repo) which branches/tags of the repo are included in sphinx-multiversion? We normally have lots of branches with some experimental stuff that we don't need to build a documentation for...

Thank you very much for the fixes and explanations! Seems to work now :-)

Great!

Do you see an easy method to define (e.g. in a file in the repo) which branches/tags of the repo are included in sphinx-multiversion? We normally have lots of branches with some experimental stuff that we don't need to build a documentation for...

https://holzhaus.github.io/sphinx-multiversion/master/configuration.html#tag-branch-remote-whitelists


Autopilot code location?

Hi,
I am looking for the hard coded AI that is used by the non player characters. Is it located in /carla/Unreal/CalraUE4/Plugins/Carla/Source/Carla/Vehicle  ?
If yes, then which file is it from all of the ones in there?
I want to modify the autopilot to make it follow the RoutePlanners to reach a destination point. To do so I need to understand how your autopilot works and in particular I am trying to reach the point in the autopilot code where it randomly decides which route to take in junctions. So it would be much appreciated if you point me towards the files that I am looking for.
Thanks

It sounds to me that you could just modify the 'basic agent' associated files to perform your task. In this case, you would need to go to the file 'automatic_control.py' and see that 'BasicAgent' is selected as default. Then go to the PythonAPI/agents/navigation/basic_agent.py and modify the function def set_destination(self, location) in the option end_waypoint. You can get your desired final waypoint from the map itself.
The vehicle automatically follows the planned routes, it computes the next waypoint at every timestep, and every waypoint is located in a planned route.

Hi @lijecaru
Thanks a lot for your reply. Yes indeed that is what I was looking for. How do I set this end_waypoint from another .py script.
For example in the spawn_npc.py example it spawns the vehicles but I can't find where does it set the vehicles' destinations to. How would I be able to do that? Would it be something like set_destination(self, carla.location(x,y,z))
Thanks

Also what is the diffference between the autopilot in spawn_npc.py and the one in automatic_control.py/basic_agent.py ?
Thanks

I am not sure.  The autopilot in spawn_npc.py is implemented by the server side and following the route generated by opendrive, and it looks smoothly when running.
The one in automatic_control.py is controlled by client-side and PID controller.  It performs worsely i think.

Hi @Derekabc
Yes, I totally agree. The autopilot in spawn_npc.py follows the splines and performs really well. The one in automatic_control.py is pretty awful.
Any idea where is the code for the autopilot used in spawn_npc.py i.e. the one that follows the splines?
Thanks

Thanks @Derekabc,  but that doesn't show any of the AI used or how it follows the way points in the route planner...?
What does the file /carla/Unreal/CarlaUE4/Plugins/Carla/Source/Carla/Vehicle/WheeledVehicleAIController.cpp  do then?

Also what is the diffference between the autopilot in spawn_npc.py and the one in automatic_control.py/basic_agent.py ?
Thanks

Do you know the difference now? I have the same question...

I have the issue, can anybody please guide me how to assign a specific destination to the vehicle? where it must stop?

@SamiaAbid  You can check the location of the vehicle and set the control.steer = 0, control.throttle = 0 and control.brake = -1 if the location meets the requirement.


Mobile Metamask

login popup is no longer shows up on mobile metamask, but directly logs in
automatically hides the "waiting for you to approve" modal after 5 seconds if environment is Metamask app.
adding vConsole for easier debugging on mobile devices

corresponding issue: https://github.com/OriginProtocol/origin-dollar/issues/129
The second workaround is required because the Metamask app has a bug where the contract call does not throw an exception when user rejects the transaction. The javascript execution is just stuck on the await line calling the contract function. Interestingly if you quit the app and re-enter immediately, the correct error with "user rejected the transaction..." still gets thrown.
The other interesting error I've found is that in 60% of the cases on my android (Oneplus 8Pro) the metamask app wouldn't inject the provider to window object (window.ethereum). Not reproducible on the iPhone, but happens constantly on Android. The dapp login doesn't work of course when that happens, and there is not much we can do about it.

Nice work, and good to know about the issues that are out of our control, thanks 🦊

@sparrowDom it might be good to create corresponding issues at https://github.com/MetaMask/metamask-mobile or link to existing ones. I did a quick search and didn't find any that I thought would match.

Just for tracking purposes found a similar issue with window.ethereum being undefined and left a comment here.
And reported a separate issue regarding the rejected transactions not throwing errors.


Fix error if malloc fails

It's not very likely, but in case you're running out of memory malloc might fail and this change prevents that we're overwriting arbitrary data.

Good fix!


Use dl-cdn alpine mirror by default

... and get rid of random mirror. It's expected that cdn chooses the nearest mirror by default. So it should be more reliable than picking a random one. (Note: here were no CDN mirror at the inception of vagga)

Works fine as far as I can see.


Create an NgGrid Component

There should be a component option as well as the directive currently provided. This should take the basic form of:
<ngGrid [items]="items"></ngGrid>

Things that need to be considered:

Event bindings/triggers
Serialisation #66
Custom item rendering
Probably more...

From @tudorgergely https://github.com/BTMorton/angular2-grid/issues/80#issuecomment-221345908

So I played around to the component and ended up with a template like this (inside it):
   <div [ngGrid]="config">
       <div [ngGridItem]="item.config"
            *ngFor="let item of items; let i = index"
            id="gridItem_{{i}}">
       </div>
   </div>

and the template of a gridItem is given by another component (and added dynamically):
private addItems(items: any[]): void {
items.forEach((item, index) => this.createItem(item.component, index));
}
private createItem(component, index) {
this.cmpResolver.resolveComponent(component)
.then((factory:ComponentFactory) => factory.create(this.viewContainer.injector, undefined, #gridItem_${index}))
}
This way we can specify the component and config of our item inside an object with this structure:
   {
       config: {
           col: 0,
           row: 12,
           sizex: 16,
           sizey: 2,
       },
       component: TestComponent,
   },

What do you think of this approach? it is working pretty well for me

Actually, I think it would be better to extend the NgGrid directive. That way for things that can only be controlled by the component itself (dragging in external items/dragging between grids) the new/extra code can be added explicitly to the component, rather than having to try doing various checks on the grid directive. Thoughts?

I would not extend the NgGrid directive. What I would do is extract a lot of the things NgGrid does in separate services and reuse that.
Extending things is not really nice and good, I would favor composition (through services in this case) to extension everyday.
Don't you agree?

Not perfect.
https://github.com/lunzhang/ng2-grid

I would like to see such a component. If you want to access some of the NgGrid functions there is no way to do it.

Hello @BTMorton , Any Progress on this feature ?


ESO broke after GKE upgrade

Hi, I upgraded our GKE cluster from 1.21.* to 1.22.* with ESO version 5.2.
After upgrade immediately ESO went down and started restarting, after some investigation, one of the CRDs is unreachable "clusterexternalsecrets.external-secrets.io". From the controller and kubectl.
Controller log:
failed to list *v1beta1.ClusterExternalSecret: Internal error occurred: error resolving resource
k get clusterexternalsecrets.external-secrets.io
Error from server (InternalError): Internal error occurred: error resolving resource
I have tried upgrading ESO all the way to 0.7.2, but the resource is stuck and I can not delete it for a full reinstall of ESO.
I didnt have any clusterExternalSecrets either ..
Before I did the same upgrade on almost identical GKE testing cluster and no problems appeared.

finalizer was blocking deletion.. :sweat:

I had the same issue.
Is it a known behaviour?
Is the upgrade the path to fix it?


Fix Versioning Regression

Fixes #3525.

:x: There is undocummented changes. Run chronus add to add a changeset or click here.
The following packages have changes but are not documented.

:x:@typespec/versioning

Show changes

You can try these changes at https://cadlplayground.z22.web.core.windows.net/prs/3526/
Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/prs/3526/


secure as a CookieOption

Feature request
Is your feature request related to a problem? Please describe.
I want to have my sameSite attribute set to none but this is not happening because CookiesOption & GoTrueClient Api doesn't have a option to set secure attribute.
Describe the solution you'd like
An option to set secure attribute
Idk if its already implemented or there is even a work around, if there is any please share. Thanks for help in advance 🙏

This will be very helpful in passing cookies to the server after 'oAuth' call back.
Currently, 'same-site' is blank. With recent changes, browser treat this as 'lax' and hence are not sending it to cross domain server.
As of now, 'same-site' can be 'none' only if it is 'secure'.
Without this feature, an extra trip is required from the client side to send the cookies to the server.

While peeking into the code, I realise that the cookie setting for oauth callback is done at gotrue repo.
Hence, the oAuth issue that I had raised belongs to gorue repo. Will file a separate ticket there.

We no longer recommend the use of sb-access-token and sb-refresh-token cookies. Please read our latest server side rendering document to understand best practices for accessing access and refresh tokens on the server.


GET arlas/explore/_list  in error

In gitlab by @sylvaingaudan on Jul 21, 2017, 11:06
GET arlas/explore/_list is in error when arlas is started for the first time and that its .arlas index does not exist yet
It should return an empty array instead.

In gitlab by @sylvaingaudan on Jul 21, 2017, 12:09
changed milestone to %7

In gitlab by @sfalquier on Jul 21, 2017, 14:07
added ~1918367 label

In gitlab by @sfalquier on Jul 21, 2017, 15:38
added ~1918368 and removed ~1918367 labels

In gitlab by @sfalquier on Jul 21, 2017, 16:20
mentioned in commit 8255ef26ab505be03573f2827aa05fdbd3e6c91c

In gitlab by @sfalquier on Jul 21, 2017, 16:21
mentioned in merge request !70

In gitlab by @sfalquier on Jul 21, 2017, 16:25
mentioned in commit cd3d0db768f7033fef2ba11b36fb2b71de882836

In gitlab by @sylvaingaudan on Jul 21, 2017, 17:09
closed via merge request !70

In gitlab by @sylvaingaudan on Jul 21, 2017, 17:20
mentioned in commit 5037c4c02cca64b577dbfd22bc88a2f2094a6a40


How to recover transaction sent without Memo ID

Please provide feedback on how to recover a transaction of XLM sent from Coinbase to BitMart without a Memo ID included.
Thanks

#im in same fuckin boat. I pretty much lost $100

Has anyone received a response when emailing for help?

No I have not received a response yet . I need assistance
Get Outlook for iOShttps://aka.ms/o0ukef

From: Melikey22 @.>
Sent: Tuesday, April 27, 2021 9:10:10 PM
To: stellar-expert/stellar-expert-explorer @.>
Cc: jorgedenas @.>; Comment @.>
Subject: Re: [stellar-expert/stellar-expert-explorer] How to recover transaction sent without Memo ID (#420)
Has anyone received a response when emailing for help?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/stellar-expert/stellar-expert-explorer/issues/420#issuecomment-828087299, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ATYS424IVJ75QES65XGG3CLTK5VAFANCNFSM43IUIOVA.


Integrate with HASS Local Calendar

Request
Hi there,
I have been happily using the Afvalwijzer for a while now. Great work so far!
As a side note: This is my very first feature request. So be gentle. ;)
In the 2023.1 release of HomeAssistant the 'Local Calendars' integration was introduced. With the first (minimal) functionality made available via the UI.
Now with 2023.2 support is added to also add events with a service 'calendar.create_event'.
Would it be an idea to also integrate the afvalwijzer dates into a local calendar?
This creates the possibility to:

Calendar view: View the upcoming garbage collections in the Calendar view
Notifications: Use the calendar events for generic notification automations
Easy representation: I can imagine there will also be other cards/views made available for the Local calendars. By integrating Afvalwijzer those events can also tag along.

Very curious to know how others feel about this.
From a technical point of view, I am not very sure how big the impact would be to make this happen. So any replies on the complexity would be more than welcome.
Cheers!
Bart
Additional information
No response

Sorry, I have been quiet busy. I wlll respond asap!


Heavy CPU usage (2)

This might be related to https://github.com/Microsoft/vscode-python/issues/2149
I noticed a python process running at high cpu (80-90%) for a long time (at least 2 minutes) when I'm editing the code. After a few acts of commenting lines in and out I was able to judge that this CPU jump was caused by a tqdm call.
Here is a mwe reproducing the problem for me:
from tqdm import tqdm

x = 0
for i in tqdm(range(10)):
    x += i

print(x)

If I remove the tqdm call and save the file CPU usage drops instantly:
from tqdm import tqdm

x = 0
for i in (range(10)):
    x += i

print(x)

Environment data

VS Code version: 1.29.1
Extension version (available under the Extensions sidebar): 2018.11.0
OS and version: OS X 10.11.6
Python version (& distribution if applicable, e.g. Anaconda): 3.6.6, Anaconda
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): conda
Relevant/affected Python packages and their versions: tqdm 4.28.1

Expected behaviour
No heavy CPU usage
Actual behaviour
Heavy CPU usage
Steps to reproduce:

Type in the code above
Save

I have a similar issue, that started recently. Language server consumer CPU and does not respond to any UI "actions" (goto, usages, etc.). Language server definitely worked fine with my project a couple of weeks ago. But now it stopped working.
How can I troubleshoot?

I have the following in "Output":
Parsing complete. Waiting for analysis entries to enqueue.
Enqueue complete. Waiting for analysis to complete.
Waiting for parsing to complete.
Parsing complete. Waiting for analysis entries to enqueue.
Enqueue complete. Waiting for analysis to complete.

and that's it.

@SiLiKhon Are you using the language server or Jedi?

I'll have to watch out for this to get actual logs, but I definitely remember seeing multiple "parsing","analysis" and "module reload" messages popping up in the Output panel (opening the panel if I'd had it closed).
Closest incident I remember was for "module reload" when editing Notebook files from the Jupyter interface while they were in my workspace. I excluded .ipynb and so far that exact scenario hasn't happened again.

@SiLiKhon Are you using the language server or Jedi?

I'm using Jedi. Sorry, I did not realize there were different tools when opening the issue. I guess, the resolution here is same as in https://github.com/Microsoft/vscode-python/issues/2149.
From https://github.com/Microsoft/vscode-python/issues/2344 it seems that language server is not supported for my OS version :(

Looks like it's actually a pylint issue. Just running it from terminal on a script file with the above code has the same effect.

I'm using language server. By the way, how can I install specific version of language server? Want to know if the problem started for me with latest releases (0.1.65+) or not.

@SiLiKhon Pylint issues we can't help with.
@yevhen-m we don't support downloading specific versions. You can use "python.analysis.downloadChannel": "daily" if you want bleeding edge. Otherwise please report issues to github.com/microsoft/python-language-server .

And FYI I don't know what is creating the
Parsing complete. Waiting for analysis entries to enqueue.
Enqueue complete. Waiting for analysis to complete.
Waiting for parsing to complete.
Parsing complete. Waiting for analysis entries to enqueue.
Enqueue complete. Waiting for analysis to complete.

output as our extension doesn't do that.


Showing black screen for some seconds

Hi,
I'm using twilio programmable video in my iOS Application. After the Call connected, it shows the black screen for 15-20 seconds. I tried to resolve this issue but I couldn't find the appropriate solution. Can anyone give a valid solution.
   func subscribed(to videoTrack: TVIRemoteVideoTrack, publication: TVIRemoteVideoTrackPublication, for participant: TVIRemoteParticipant) {
          if !participant.remoteVideoTracks.isEmpty {
                participant.remoteVideoTracks[0].remoteTrack?.addRenderer(self.getRemoteView())
           }
  }

Hi @ceaglest @piyushtank  This is my Room Sid ->  RMa984beebdaafb5260ea423f04915194c . Can you check it why the audioTrack is not working.

@sathishvgs We will look it up the Room's traffic however it will be helpful debug the problem if you could answer Chris' questions asked in his last reponse  -

Audio should not be enabled and disabled unless you explicitly toggle the enabled property of the Track. Is there something in your application code that modifies the property?

It would be great if you could let us know if this issue is reproducible in the QuickStart app (especially with the latest 2.0.0-preview9 release). As a reference application, it handles all audio related delegate callbacks using best practices.

Hi @piyushtank ,
I tried in QuickStart app, it worked fine. In my application, I'm trying with multi participant call and following the same for localAudioTrack. Can you show the logs for the above Room sid.

Hey @sathishvgs,
Thanks for the Room SID. I pulled the traffic and there are a couple of things to note:
Summary for room name: test | sid: RMa984beebdaafb5260ea423f04915194c | account sid: ACxxx
Topology: group | Adhoc: True | TURN: True | Recording: False | Video Codecs: VP8 | Media Region: us1
VMS sid: VMxxx
Participants:
P1: name: c5dd528a-5ac9-41d6-9650-2fcf21f3d712 | sid: PA83a1904ef34b47e85ecf0a7f1e042d4e | call sid: CAeb2b9251f0be5b7d5b25fbcca101e8d7 | sdp format: Plan B
P2: name: 45b35fc7-c2f3-41d7-9e2a-d0609c6a2aad | sid: PA0431a7cca683f8a8c2c990980027ba25 | call sid: CAe79cd03b6d30e0e227732aa2a8dc7377 | sdp format: Plan B

I can see in the logs that P1 performs a sync. You may have noticed that media did not return after this, which is due to https://github.com/twilio/video-quickstart-swift/issues/154. After syncing P1 also mutes/disables their video. Are you toggling the LocalVideoTrack.enabled property here?

Once P2 connects the Participants subscribe to each other's Audio and Video tracks pretty quickly.

Several MB of Data is exchanged between the Participants.

Did you hear audio only for the first portion of the Room, and not after the sync event, or never at all?

If you're outside of North America you should consider using our Media Regions API to select a media server nearer to your Participants. You can read up on creating Rooms via REST here and see a list of the regions that we support here.

2018-02-03T09:47:48.361Z: P1 sent connect with audio + video tracks
2018-02-03T09:47:48.361Z: P1 sent offer for peer connection id 90f4ca95-e65b-ccb7-b76a-c4cbbe05b530 revision: 1
2018-02-03T09:47:48.362Z: [add_labeled_room]
2018-02-03T09:47:48.364Z: [room_created]
2018-02-03T09:47:49.246Z: P1 received connected
2018-02-03T09:47:49.246Z: P1 received answer for peer connection id 90f4ca95-e65b-ccb7-b76a-c4cbbe05b530 revision: 1
2018-02-03T09:47:49.246Z: P1 published video track name: 41869becd8ca91470469d1e5ed6905457b9f sid: MTe268d116e23de72b77f7b3973c4ca348 revision: 1
2018-02-03T09:47:49.246Z: P1 published audio track name: a11ca5f9148f94f3385ffbddc2a35bb8912e sid: MT313eb6d1b2f05aa9f404baab0389b27f revision: 1
2018-02-03T09:47:50.255Z: P1 sent ice complete for peer connection id 90f4ca95-e65b-ccb7-b76a-c4cbbe05b530 with 3 ice candidates [ 2 host, 1 srflx, 0 relay ] revision: 4
2018-02-03T09:47:51.314Z: [ice_connection_state_change] for P1 with state CONNECTED
2018-02-03T09:47:51.314Z: [ice_connection_state_change] for P1 with state COMPLETED
2018-02-03T09:48:17.076Z: P2 sent connect with audio + video tracks
2018-02-03T09:48:17.076Z: P2 sent offer for peer connection id f6aB1fa7Bbee74D2B0D05da19E451e1f revision: 1
2018-02-03T09:48:17.095Z: P2 received connected
2018-02-03T09:48:17.095Z: P2 received answer for peer connection id f6aB1fa7Bbee74D2B0D05da19E451e1f revision: 1
2018-02-03T09:48:17.095Z: P2 published video track name: micvideo sid: MTc9a81cebf468aad0a96cbbdd699f5470 revision: 1
2018-02-03T09:48:17.095Z: P2 published audio track name: ECABf8e1D1A9DBFe423f28B65EE74636 sid: MTda10801deba5371729da745e4817a171 revision: 1
2018-02-03T09:48:17.096Z: P1 received offer for peer connection id 90f4ca95-e65b-ccb7-b76a-c4cbbe05b530 revision: 2
2018-02-03T09:48:17.096Z: P1 subscribed to track sid: MTda10801deba5371729da745e4817a171 revision: 2
2018-02-03T09:48:17.096Z: P1 subscribed to track sid: MTc9a81cebf468aad0a96cbbdd699f5470 revision: 2
2018-02-03T09:48:17.096Z: P2 received offer for peer connection id f6aB1fa7Bbee74D2B0D05da19E451e1f revision: 2
2018-02-03T09:48:17.096Z: P2 subscribed to track sid: MTe268d116e23de72b77f7b3973c4ca348 revision: 2
2018-02-03T09:48:17.096Z: P2 subscribed to track sid: MT313eb6d1b2f05aa9f404baab0389b27f revision: 2
2018-02-03T09:48:17.929Z: P2 sent ice complete for peer connection id f6aB1fa7Bbee74D2B0D05da19E451e1f with 3 ice candidates [ 1 host, 1 srflx, 1 relay ] revision: 4
2018-02-03T09:48:17.929Z: P2 sent answer for peer connection id f6aB1fa7Bbee74D2B0D05da19E451e1f revision: 2
2018-02-03T09:48:20.473Z: [ice_connection_state_change] for P2 with state CONNECTED
2018-02-03T09:48:21.364Z: P1 sent answer for peer connection id 90f4ca95-e65b-ccb7-b76a-c4cbbe05b530 revision: 2
2018-02-03T09:48:21.371Z: [ice_connection_state_change] for P1 with state CONNECTED
2018-02-03T09:48:30.638Z: [ice_connection_state_change] for P2 with state COMPLETED
2018-02-03T09:48:34.072Z: [ice_connection_state_change] for P1 with state COMPLETED
2018-02-03T09:49:20.627Z: P1 sent sync
2018-02-03T09:49:20.627Z: P1 sent ice complete for peer connection id 90f4ca95-e65b-ccb7-b76a-c4cbbe05b530 with 3 ice candidates [ 2 host, 1 srflx, 0 relay ] revision: 4
2018-02-03T09:49:20.627Z: P1 sent answer for peer connection id 90f4ca95-e65b-ccb7-b76a-c4cbbe05b530 revision: 2
2018-02-03T09:49:20.629Z: P1 received synced
2018-02-03T09:49:20.629Z: P1 received offer for peer connection id 90f4ca95-e65b-ccb7-b76a-c4cbbe05b530 revision: 2
2018-02-03T09:49:20.935Z: P1 sent offer for peer connection id 90f4ca95-e65b-ccb7-b76a-c4cbbe05b530 revision: 3
2018-02-03T09:49:20.943Z: P1 received answer for peer connection id 90f4ca95-e65b-ccb7-b76a-c4cbbe05b530 revision: 3
2018-02-03T09:49:22.424Z: [ice_connection_state_change] for P1 with state CONNECTED
2018-02-03T09:49:22.712Z: [ice_connection_state_change] for P1 with state COMPLETED
2018-02-03T09:49:22.744Z: P1 sent ice complete for peer connection id 90f4ca95-e65b-ccb7-b76a-c4cbbe05b530 with 5 ice candidates [ 2 host, 1 srflx, 2 relay ] revision: 6
2018-02-03T09:49:49.211Z: P1 sent update with track changes: video was paused revision: 4
2018-02-03T09:49:49.215Z: P1 published video track name: 41869becd8ca91470469d1e5ed6905457b9f sid: MTe268d116e23de72b77f7b3973c4ca348 revision: 2
2018-02-03T09:49:49.215Z: P1 published audio track name: a11ca5f9148f94f3385ffbddc2a35bb8912e sid: MT313eb6d1b2f05aa9f404baab0389b27f revision: 2
2018-02-03T09:49:49.925Z: P1 sent disconnect
2018-02-03T09:49:49.928Z: P1 received disconnected
2018-02-03T09:49:49.930Z: [group_billing_event] for P1 with bytes sent 3200607, bytes received 11885346, recording time in secs 0
2018-02-03T09:49:49.930Z: P2 received offer for peer connection id f6aB1fa7Bbee74D2B0D05da19E451e1f revision: 3
2018-02-03T09:49:50.498Z: P2 sent disconnect
2018-02-03T09:49:50.500Z: P2 received disconnected
2018-02-03T09:49:50.503Z: [group_billing_event] for P2 with bytes sent 6450078, bytes received 2910135, recording time in secs 0
2018-02-03T09:49:50.506Z: [delete_labeled_room]

Regards,
Chris

@ceaglest Thank you for your kindly help. I have fixed my issue. It happened because of the callkit. Now, its working fine.

Great to hear @sathishvgs! I'll close out this ticket if everything is working for you.

@sathishvgs  - I'm having similar problems with a black video screen being shown.  What was the change that you made that fixed the problem?

Hi @adamrhunter
For me, Audio Track problem is only fixed. Black video screen is still showing.

I'm having similar issue. Black Screen appears even after participant video track delegates are called. What would be the issue?
It's happening in android and iOS devices. There is a huge delay after the participant connected.
In iOS, it's taking more time (10 sec -2 mins) to display participant video (participant is already joined from the android device)
Reference:

More details:
User 1 - Codec: VP8
User 2 - Platform: iOS SDK Version: TwilioVideo', '~> 3.2.3' and com.twilio:video-android:5.5.0 codec: ios: H246 and android: VP8
In User 2, User 1 video is coming blank and sometime taking more time to display as well. Will there any conflicts with different SDK Version?
log.txt
@ceaglest can you please help here. Any workaround for this?

Similar issue related to android:
ios - 536

Similar issue related to RN:
RN - 94

Please open this ticket! This is not yet resolved.


How to login?

I connected to server that requires authentication. Raw output is like this:
Trying <IP_ADDRESS>...
Connected to <IP_ADDRESS>.
Escape character is '^]'.
Ubuntu 16.04.2 LTS
wapmorgan-10HYS04800 login:

How to authentication with your library?
I tried this, but it hangs up for a while:
$telnet = Graze\TelnetClient\TelnetClient::factory();
$telnet->connect('<IP_ADDRESS>:23');
var_dump($telnet->execute('wapmorgan', 'wapmorgan-10HYS04800 login:'));

The second param you pass to execute is the prompt to expect after the command is sent.
For your example to work, the raw output would have to look something like:
Trying <IP_ADDRESS>...
Connected to <IP_ADDRESS>.
Escape character is '^]'.
Ubuntu 16.04.2 LTS
wapmorgan-10HYS04800 login:
wapmorgan <--- the username you sent
wapmorgan-10HYS04800 login: <--- the response you're expecting

In reality, I expect the raw output to be along the lines of:
Trying <IP_ADDRESS>...
Connected to <IP_ADDRESS>.
Escape character is '^]'.
Ubuntu 16.04.2 LTS
wapmorgan-10HYS04800 login:
wapmorgan <--- the username you sent
password: <--- the response you're expecting

In which case your first command should look like:
$telnet = Graze\TelnetClient\TelnetClient::factory();
$telnet->connect('<IP_ADDRESS>:23');
var_dump($telnet->execute('wapmorgan', 'password:'));

For some reason it' not working as expected.
php > require 'vendor/autoload.php';
php > $telnet = Graze\TelnetClient\TelnetClient::factory();
php > $telnet->connect('<IP_ADDRESS>:23');
php > var_dump($telnet->execute('wapmorgan', 'Password:'));

And the raw output of success login is:
Trying <IP_ADDRESS>...
Connected to <IP_ADDRESS>.
Escape character is '^]'.
Ubuntu 16.04.2 LTS
wapmorgan-10HYS04800 login: wapmorgan
Password:
Last login: ***
Welcome to Ubuntu 16.04.2 LTS (GNU/Linux 4.8.0-53-generic x86_64)

 * Documentation:  https://help.ubuntu.com
 * Management:     https://landscape.canonical.com
 * Support:        https://ubuntu.com/advantage
...

Is the program still hanging for a while now, or exiting immediately?
To debug, it's useful to see what is being returned to the client. You can do this by var_dump-ing $buffer after this line https://github.com/graze/telnet-client/blob/master/src/TelnetClient.php#L229
(This will produce a lot of output!)

_I changed PC, so hello message slightly differs.

Ok, discovered that the problem is: PromptMatcher::isMatch() checks that prompt string ends with a newline-char. (why?)
And also, this line
$pattern = sprintf('/%s%s$/', $prompt, $lineEnding);

I think it's useful when you contact with SMTP, where response messages ends with \n char. But when it's used as a remote terminal connection it's not the expected behavior.

Removed these check and pattern creation and now it works.

I've not seen that behaviour before; each server I've used this client with sends the newline char at the end of any transmission, to signal that transmission is complete.
Is the newline omitted in all responses, or just the Password:  prompt?
I've just seen your PR, I'll take a look separately.

That's behavior of openbsd-inetd 0.20091229-2ubuntu3.

It looks like a bad assumption on my behalf, based on embedded systems I've been connecting to.
Closing this and moving to #10.

Good day, i have problem when using this on php. here is the error message
Type: Graze\TelnetClient\Exception\TelnetException
Message: unable to create socket connection to [XXX.XXX.XX.XX:XXXX]
Filename: C:\xampp\htdocs\pnbsb-backend\vendor\graze\telnet-client\src\TelnetClient.php

Hi @chesterLV19, that appears to be a networking issue rather than an issue with this library.
Ensure the environment that's running the code is able to connect to XXX.XXX.XX.XX:XXXX.
You can test using the telnet command from Windows cmd: telnet XXX.XXX.XX.XX XXXX

thank you for your answer, i'll try this :)
On Thu, Sep 6, 2018 at 12:47 AM John Smith<EMAIL_ADDRESS>wrote:

Hi @chesterLV19 https://github.com/chesterLV19, that appears to be a
networking issue rather than an issue with this library.
Ensure the environment that's running the code is able to connect to
XXX.XXX.XX.XX:XXXX.
You can test using the telnet command from Windows cmd: telnet
XXX.XXX.XX.XX XXXX
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/graze/telnet-client/issues/9#issuecomment-418800518,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AbG9CUIpmLlhDlP93Iub9Ecsx70CULBtks5uYAADgaJpZM4N7CHh
.

--
Chester L. Viñas
Program Manager/Developer
NMGResources Inc.


Update kubeconfig file write to flag sensitive content

This will suppress the CLI output of the kubeconfig file. Useful when run by CI/CD and some may have access to pipeline jobs.
PR o'clock
Description
Please explain the changes you made here and link to any relevant issues.
Checklist

[ ] terraform fmt and terraform validate both work from the root and examples/* directories
[ ] CI tests are passing
[ ] I've added my change to CHANGELOG.md and highlighted any breaking changes
[ ] README.md has been updated after any changes to variables and outputs. See https://github.com/terraform-aws-modules/terraform-aws-eks/#doc-generation

What in this file is sensitive?

I don't think anything in this file is sensitive and there's no updates in a while so I'll close this one.


Move domain check to pre-filter

In the current version, apply_block_domains checks if a domain has been blocked only after it requested info upstream. This creates a situation where the client ask for a domain that it knows its going to discard.
The change in this commit, moves the domain check in pre_filter and leaves the IP check for post_filter intact. This way, all the domains that are blacklisted by the user will be REFUSED immediately without going upstream.

This has been merged.
Thank you for your contribution!


Feature Request | Input Handler for Presto/trinodb.com

There are multiple input Handlers avilable already like PSQL or Mysql etc, But it will be nice to have one for Presto
Feature Request - Input Handler for Presto/trinodb

Would you create a PR so we can support you?

Would you create a PR so we can support you?


Added image url and onerror, onload events on callbacks

Error and success callbacks should have image url and event parameters for more detailed information purposes.

This would be really helpful, for example when Adblock Plus blocks a lazy-loaded image. I was going to propose a patch where removing the options.src attribute was done after the error call, but this would be much cleaner.

Fixed in v. 1.5.3


Room destroy throws error when httpd is false

e.g. (coffeescript)
  beforeEach ->
    @room = helper.createRoom httpd: false

  afterEach ->
    @room.destroy()

Throws TypeError: Cannot read property 'close' of undefined.
I think because .destroy is just a proxy for @robot.server.close() which maybe shouldn't be run if there was no httpd server created? Just guessing, sorry I didn't have time to contribute more.

Thank you for your reporting!
I'll investigate about that but I din't have time too so it may take a while 😓

should fix it. @timkinnane Could you check on your side?

Thanks @pchaigno, your changes look good, but can you give me a hint how to test it. I tried npm installing the repo from github, but it seems that it needs to be compiled by the npm publish process on a proper version release to work via npm. Are you just cloning and running the prepublish script locally then requiring the path?

You can temporarily replace the version for hubot-test-helper in your package.json file with my repository and branch:
"hubot-test-helper": "pchaigno/hubot-test-helper#fix-room-destroy-httpd",

I tried doing that with my own fork and got an error from npm install, because No version provided in package.json, so I added the version to my fork's package.json, but then found that the /src folder is in the .npmignore so it can't be used as a node module from the repo. I think it needs to be cloned and built locally, I was just trying to avoid the bother. :/


Unity HelloWorld example

When I look at a DI container for unity, the first thing I am hoping to find in the documentation is a hello world example that explains all the steps needed to register an IHelloService dependency and then resolve it from within a unity script.
I quickly went through the Zenject docs and there's a lot of documentation on why DI is good which I already know and is not what I am looking for. In fact, it's making it harder for me to find the information I really care about: how do I get DI working in unity with Zenject.
It would be great to have such a hello world example and Zenject specific docs for people who already know about DI and just want to learn how to use Zenject :)

There is the Hello World Example section.  Is that what you're looking for?

I only see dependency registration in that example; It's missing the resolution part. It would be good to show how to resolve a dependency from a Script.

Ah yeah true, you're right, there is no injected fields in that example.
I assume when you resolve you mean, injecting dependencies by using [Inject] fields, constructor parameters, etc. ?
Or do you mean directly by calling Container.Resolve?

Yes, I meant injecting. I eventually figured it out from the rest of the docs but having it part of the Hello World example would have saved me some time :)

Yes I agree.  I'll change it to inject the "Hello World" string into that class.

Sounds good, thanks!


Create project command

Hi there,
Is it possible to do something like composer create-project ....?
If I do this currently it gets the old packages that were on packagist but not the latest version. When I do composer require it is getting the latest version so that is working. Perhaps I'm missing something else.. 🤔

Hi,
Did you use the --repository= option?
I tried to reproduce this error, but our instance installs the latest version of the package.


update

Coverage remained the same at 97.866% when pulling 8bceb8f33261b76719fcdcb9831213febc82b7eb on djsteph888:master into be15f1c742b35134ce5bd35521d8bf1ab1ba67cf on thelinmichael:master.