Flutter templates contain a lot of IntelliJ files

The Flutter templates contain a lot of IntelliJ files, e.g.:
drwxr-xr-x   5 mit  primarygroup   160B Sep 14 12:05 .idea
-rw-r--r--   1 mit  primarygroup   896B Sep 14 12:05 testmod.iml
-rw-r--r--   1 mit  primarygroup   1.4K Sep 14 12:05 testmod_android.iml

These have no utility to a developer using another editor/IDE (such as vscode), or using the terminal. Should we consider only creating these when creating projects from within IntelliJ/Android Studio?

cc @devoncarew, WDYT?

It may be possible to move the foo_android.iml inside the android directory; I - @stevemessick - would have to look into it.

These have no utility to a developer using another editor/IDE (such as vscode)

Agreed. What I want to make sure is that there's as little friction or issues as possible with opening the project w/ Android Studio. It's generally not as easy to open a new project with IntelliJ based tools as I'd like, so doing what we can to reduce friction here is worthwhile.
A while ago we did put some code into IntelliJ to create missing project files when necessary - we'd have to verify that this handles all anticipated use cases (that the metadata files really are optional).

A quick experiment indicates that, yes, you could move the *_android.iml file into the android directory, as long as .idea/modules.xml exists and is changed to point to the new location. You cannot move the other *.iml file, though. I'm not sure if that is because something in the Flutter plugin expects it to be at top-level (and could be changed) or if it is something IntelliJ depends on.

Did we decide on a solution here?

I'd say we should move the foo_android.iml inside the android/ directory, but that may now be more trouble than it's worth. The name of the directory can now be either .android/ or android/, and if we move where the file is, the make-app-editable command would have to know to update the references to the .iml file from the .idea/modules.xml file.
I'd like to close this as won't do, or drop it to a P3.

I agree that moving the file has little value to IntelliJ developers. However, I still question the value of 7 extra IntelliJ metadata files for developers that use terminal or vscode. So could we not consider creating these only during project creation in an IntelliJ-based IDE?

Possibly this is the same as #14723?

This was addressed in https://github.com/flutter/flutter/pull/24311. Whatever is left is a dup of #14723.


KeyboardDatePicker accessibility issues - multiple buttons missing labels

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
The KeyboardDatePicker calendar and month buttons are announced as 'unlabeled button" when using a screen reader like JAWS. A screen reader user has no context for the purpose of these buttons if they are unlabeled.

Expected Behavior 🤔
All buttons should have an aria-label or title attribute for Assistive Technology users.

Steps to Reproduce 🕹

https://qm22m.csb.app/
Steps:
Run thru the following steps with a screen reader like JAWS running. If you're not using a screen reader then inspect each button with the browser dev tools and confirm button is missing an aria-label or title attribute.

Press Tab to set focus to the Calendar icon button.
Result: Calendar Button announces as "Unlabeled Button"
While on the Calendar Button press enter to open the Calendar dialog.
Press Tab to set focus to the Previous Month button
Result: Previous Month button announces as "Unlabeled Button"
Press Tab to set focus to the Next Month button
Result: Next Month button announces as "Unlabeled Button"

Context 🔦
The missing labels on buttons makes it hard for Assistive Technology users to use the Calendar dialog.
Running the Calendar Component thru Microsoft's Accessibility Insights tool provides the following failure details for the buttons.
Title: WCAG 4.1.2: Ensures buttons have discernible text (.MuiPickersCalendarHeader-iconButton.MuiIconButton-root:nth-child(3))
Tags: Accessibility, WCAG 4.1.2, button-name
Issue: Ensures buttons have discernible text (button-name - https://dequeuniversity.com/rules/axe/4.0/button-name?application=msftAI)
Target application: Datepicker - @material-ui/pickers component - https://material-ui-pickers.dev/demo/datepicker
Element path: .MuiDialogContent-root > .MuiPickersBasePicker-container > .MuiPickersBasePicker-pickerView > div:nth-child(1) > .MuiPickersCalendarHeader-switchHeader > .MuiPickersCalendarHeader-iconButton.MuiIconButton-root:nth-child(3)
Snippet:
How to fix:
Fix any of the following:
Element does not have inner text that is visible to screen readers
aria-label attribute does not exist or is empty
aria-labelledby attribute does not exist, references elements that do not exist or references elements that are empty
Element's default semantics were not overridden with role="presentation"
Element's default semantics were not overridden with role="none"
Element has no title attribute or the title attribute is empty
Environment: Microsoft Edge version 85.0.564.44
====
This accessibility issue was found using Accessibility Insights for Web 2.21.1 (axe-core 4.0.1), a tool that helps find and fix accessibility issues. Get more information & download this tool at http://aka.ms/AccessibilityInsights.
Title: WCAG 4.1.2: Ensures buttons have discernible text (.MuiIconButton-root)
Tags: Accessibility, WCAG 4.1.2, button-name
Issue: Ensures buttons have discernible text (button-name - https://dequeuniversity.com/rules/axe/4.0/button-name?application=msftAI)
Target application: Datepicker - @material-ui/pickers component - https://material-ui-pickers.dev/demo/datepicker
Element path: .MuiFormControl-root.MuiTextField-root:nth-child(3) > .MuiInputBase-adornedEnd.MuiInput-root.MuiInput-underline > .MuiInputAdornment-positionEnd.MuiInputAdornment-root > .MuiIconButton-root
Snippet:
How to fix:
Fix any of the following:
Element does not have inner text that is visible to screen readers
aria-label attribute does not exist or is empty
aria-labelledby attribute does not exist, references elements that do not exist or references elements that are empty
Element's default semantics were not overridden with role="presentation"
Element's default semantics were not overridden with role="none"
Element has no title attribute or the title attribute is empty
Environment: Microsoft Edge version 85.0.564.44
====
This accessibility issue was found using Accessibility Insights for Web 2.21.1 (axe-core 4.0.1), a tool that helps find and fix accessibility issues. Get more information & download this tool at http://aka.ms/AccessibilityInsights.

Your Environment 🌎
This issue is not specific to our environment and occurs on the material-ui-pickers demo site. However, this issue has been reported by Assistive Technology users in our environment.
Screen shots of Accessibility Insights tool flagging buttons on the demo site.

Tech
Version

@material-ui/core
v4.11.0

@material-ui/pickers
v3.2.9

React

Browser
Microsoft Edge Version 85.0.564.44, Chrome Version 85.0.4183.83

TypeScript

etc.

@alexbrandtalliance Could you try again with v4-alpha?

@oliviertassinari Tested v4-alpha and these issues are corrected. Do you know when these changes will move out of alpha?

Do you know when these changes will move out of alpha?

I think between 6 and 12 months. Once we have nailed the component, we will make it stable. In the meantime, we need the flexibility to make breaking changes. From my experience so far, that's not what's stopping most developers to use a component.

Thanks for the confirmation on v4-alpha.


wip/6.0 HHH-14642 verify Criteria - add DerbyConcatFunction

https://hibernate.atlassian.net/browse/HHH-14642
The missing of Derby specific 'concat' function was found during migration of ConcatTests during Critera verifying, however, the migration of that testing class requires the completion of https://github.com/hibernate/hibernate-orm/pull/4018.
However, another existing FunctionTests#testConcatFunction() would test this for Derby dialect.

Wait, @NathanQingyangXu, you'll have to discuss this with @beikov because he intentionally removed this in 003fddc. I don't think it's any sort of oversight.

Wait, @NathanQingyangXu, you'll have to discuss this with @beikov because he intentionally removed this in 003fddc. I don't think it's any sort of oversight.

I found we still need this function when any jdbc parameter usage is involved; otherwise I would not be able to migrate the org.hibernate.orm.test.jpa.criteria.basic.ConcatTest for Derby CI. There are other subtle thing involved but I am gonna explain on that in the other PR.

Then if that's the case, please make sure you carefully restore it to what I had on the H6 branch before the change that removed it. Please don't copy/paste stuff from main branch into H6.

(I think this was the most recent version: https://github.com/hibernate/hibernate-orm/blob/b6dc77280f478ff54a29315a342fc5d147cb6311/hibernate-core/src/main/java/org/hibernate/dialect/function/DerbyConcatEmulation.java.)

(I think this was the most recent version: https://github.com/hibernate/hibernate-orm/blob/b6dc77280f478ff54a29315a342fc5d147cb6311/hibernate-core/src/main/java/org/hibernate/dialect/function/DerbyConcatEmulation.java.)

Did you test it previously? I found the SQL it generated is invalid after testing on Derby, for instance
(cast(? as varchar) || '_1')

The correct SQL should be:
varchar(cast(? as varchar(32672)) || cast('_1' as varchar(32672)))

I will create a new testing case covering this error-prone Derby concat function rendering soon.

I did indeed test it, of course. But since at that time most of the test suite did not exist on the H6 branch then of course I might have missed something.
So if it has bugs, feel free to fix the bugs. But please don't go renaming the class back to the old name that doesn't conform to the new naming patterns in H6, or restoring any other old cruft.

But please don't go renaming the class back to the old name that doesn't conform to the new naming patterns in H6

Oh gawd, I realized you're just following other stuff that doesn't conform to the new standards. @beikov you have been naming stuff Functions when they are Emulations. :-(
Oh gawd I knoew this was going to happen. It's almost impossible to to keep this package clean.

But please don't go renaming the class back to the old name that doesn't conform to the new naming patterns in H6

Oh gawd, I realized you're just following other stuff that doesn't conform to the new standards. @beikov you have been naming stuff Functions when they are Emulations. :-(
Oh gawd I knew this was going to happen. It's almost impossible to to keep this package clean.

Thanks, now I understand. I didn't copy existing stuff blindly.

I didn't copy existing stuff blindly.

Sure, sorry, I got the wrong impression, because it was using the old naming from the other branch.

I didn't copy existing stuff blindly.

Sure, sorry, I got the wrong impression, because it was using the old naming from the other branch.

No worries. I found you have created a new PR to rename. I also learned a lot from your old implementation, especially the special case handling when only one arg is provided for concat function.

Let's move the discussion to https://github.com/hibernate/hibernate-orm/pull/4018#discussion_r645493427

Let me close this PR and move it to https://github.com/hibernate/hibernate-orm/pull/4018


[FE] Facility Selector Screen - Display facilities in alphabetical order

Why should we do this?

Large organizations like Achieve Health in Texas have a large number of facilities
Facilities are not currently displayed in alphabetical order, making it difficult to quickly or easily find a specific facility

Action Requested

Display the facilities in alphabetical order (a-z)

Acceptance Criteria

facilities appear on screen in alphabetical order (a-z)

Additional Context

Design mocks [FIGMA]

This ticket is a duplicate of work planned elsewhere. Closing on 10/04/2023.


Adding Project Links in Readme.md

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #XXXXX

The links for the Projects were not their in all the Categories. I have added them.

@muskanvk Thank you for making those changes.  This PR looks good to me now.

Congrats @muskanvk on your first pull request (PR)! 🎉
Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 📝


Pre transfer checks

Checks the following before displaying deposit address:

get router config, check if swaps are supported
get receiver channel balance
make sure receiver channel router balance + onchain balance > swapped amount

If any of the above fail, will display an error

Implemented during refactor of modal, closing

Implemented during refactor of modal, closing


Floating Label Behavior

There is a subtle difference in the way floating label behaves in Material-UI versus how it behaves in Polymer paper components.
It has to do with when the floating label decides to float.
In Polymer: label floats after you type the first character
In Material-UI: label floats on focus.
I'm not sure which one is correct. Or if there is a correct. But my app uses both libraries. Sometimes on the same page. And I would like to make it consistent.
Feature request: An attribute to control this behavior. For example:

Trying to use 2 different libraries, each having similar components, and having the expectation that they will always be the same is probably not the right thing to do. You will likely be running into inconsistencies very often.
You should try to use one library or the other for all your components, consistently.

@oliviertassinari This can be closed due specs. Label should  float on focus.

@kybarg Thanks!


Docs - Update page on creating and storing Dask DataFrames

[ ] Closes #xxxx
[ ] Tests added / passed
[x] Passes pre-commit run --all-files

This is a PR for updating the docs page on creating + storing Dask DataFrames. For context, this was driven by taking a look at google analytics and noticing this page is in the top 10 in terms of page views, but there was some outdated content on it (screenshot below for the past month):

PR looks great and it's awesome you're making the docs better!


Error pages do not return CORS header with --cors

This is very confusing. It means that if you send invalid SQL you will get back a CORS error, because the resulting 400 page cannot be accessed via JavaScript.

Demo: https://latest.datasette.io/fixtures.json?sql=select+blah


Environments not being loaded on the second run after being imported

Hello
Yesterday I imported a big collection from postman with 4 different environments and everything worked fine after importing.
Today when i started bruno the collections are visible but the environments are not showing, even though the .bru files are there.
I'm using linux debian.
Not sure what info can help you debug the issue but i can provide whatever you need

Hi, there are no environments at all in the top right corner dropdown ? And your environments with a .bru extension are placed in the environment folder at the right location ?

Sorry i created a duplicated issue, for some reason when i searched i could not find it a first.
The issue was the same mentioned here, which are the spaces in the names of the .bru files for the environments. After removing the spaces everything is ok.

Sorry i created a duplicated issue, for some reason when i searched i could not find it a first.
The issue was the same mentioned here, which are the spaces in the names of the .bru files for the environments. After removing the spaces everything is ok.


Price scale not displayed if first value is undefined

Lightweight Charts Version:
3.7.0
Steps/code to reproduce:
Display a chart with a dataset where the first value is missing.
[
    [ "2015", undefined ],
    [ "2016", "117000" ],
    [ "2018", "130000" ],
    [ "2019", "146100" ],
    [ "2020", "172900" ],
    [ "2021", "193900" ]
]

Actual behavior:
The price scale for this series is not shown.
Expected behavior:
The price scale for this series should try to pull axis information from the first-available value...and ultimately, should show the Y axis price scale.
Screenshots:

CodeSandbox/JSFiddle/etc link:
(none)

NOTE: Using a custom priceFormatter didn't solve the issue.

Interesting...
...using this data...

...the chart looks like this...

...and DOES show the right price scale if I scroll pretty much all the way over:

Please create a repro for your issue on jsfiddle for example.

Working in a quick jsfiddle...I can't repro the issue. My project is in React so maybe there are some differences there...
I'll look more into it to make a better repro.

From the first observation on your data:

the library doesn't support this format of the data (like on your screenshot) so most likely this is not what you provide to the library (probably you have a post-processing of this result)
the library doesn't accept undefined as a value for data. if you want to create a whitespace you need to omit value property
values MUST BE numbers, not strings

Probably it helps.

aha! my JSFiddle had numbers for values and so it worked
var container = document.createElement('div');
document.body.appendChild(container);

var chart = LightweightCharts.createChart(container, {
	width: 600,
	height: 300,
});

var series = chart.addBaselineSeries({
});

series.setData([
	{ time: '2018-11-19', value: undefined },
	{ time: '2018-11-26', value: 178.58 },
	{ time: '2018-12-03', value: undefined },
	{ time: '2018-12-10', value: 165.48 },
	{ time: '2018-12-17', value: 150.73 },
	{ time: '2018-12-24', value: 156.23 },
]);

with strings as values it DID REPRO the issue:
var container = document.createElement('div');
document.body.appendChild(container);

var chart = LightweightCharts.createChart(container, {
	width: 600,
	height: 300,
});

var series = chart.addBaselineSeries({
});

series.setData([
	{ time: '2018-11-19', value: undefined },
	{ time: '2018-11-26', value: '178.58' },
	{ time: '2018-12-03', value: undefined },
	{ time: '2018-12-10', value: '165.48' },
	{ time: '2018-12-17', value: '150.73' },
	{ time: '2018-12-24', value: '156.23' },
]);

...but I will fix my code 😅 👍

parseFloat() saved the day
Thanks @timocov


Send exception logs to email for users that experience incompatible profile

Updated sargon dependency, which exposes serde error message
Add the ability to send the error message to support so we can figure out what the problem is for each user affected.
In the email body, the following are provided

Device version
App version
If the error is of type CommonException we display the code for that

If it is specifically CommoneException.FailedToDeserializeJsonToValue we also provide the message and a stacktrace.

If the exception any other one, then we paste the message and the stacktrace

Screenshot

I am dragging this PR a bit further to attach the newer version of Sargon from https://github.com/radixdlt/sargon/pull/171


Failed at compilation

A clear and concise description of what the bug is.
Just cloned the project and tried running
dart parabeac.dart -p /Users/faridgarciayala/Desktop/Parabeac-Core/test/assets/SymbolTest-simp.sketch  -n test

I got the following error:
Align Generator: [ERROR] - NoSuchMethodError: The method 'addImport' was called on null.
Receiver: null
Tried calling: addImport("package:auto_size_text/auto_size_text.dart")
Project Builder: [ERROR] - ProcessException: No such file or directory
Command: flutter create /Users/faridgarciayala/Desktop/Parabeac-Core/test/assets/test
Provider: [INFO] - Thanks for trying our state management configuration that is now in Beta!
If you run into any issues please feel free to post it in Github or in our Discord!
Provider: [INFO] - Setting up the directories
Align Generator: [ERROR] - NoSuchMethodError: The method 'addImport' was called on null.
Receiver: null
Tried calling: addImport("package:auto_size_text/auto_size_text.dart")
lib/main.dart: Warning: Interpreting this as package URI, 'package:parabeac_core/main.dart'.
Unhandled exception:
FileSystemException: Cannot open file, path = '/Users/faridgarciayala/Desktop/Parabeac-Core/test/assets/test/pubspec.yaml' (OS Error: No such file or directory, errno = 2)
#0      _File.throwIfError (dart:io/file_impl.dart:635:7)
#1      _File.openSync (dart:io/file_impl.dart:479:5)
#2      _File.readAsBytesSync (dart:io/file_impl.dart:539:18)
#3      _File.readAsStringSync (dart:io/file_impl.dart:584:18)
#4      _File.readAsLinesSync (dart:io/file_impl.dart:590:36)
#5      PBFlutterWriter.submitDependencies (package:parabeac_core/generation/generators/writers/pb_flutter_writer.dart:71:38)
#6      GenerationConfiguration._commitDependencies (package:parabeac_core/generation/generators/value_objects/generation_configuration/pb_generation_configuration.dart:155:14)
#7      GenerationConfiguration.generateProject (package:parabeac_core/generation/generators/value_objects/generation_configuration/pb_generation_configuration.dart:122:11)

#8      ProviderGenerationConfiguration.generateProject (package:parabeac_core/generation/generators/value_objects/generation_configuration/provider_generation_configuration.dart:30:17)
#9      FlutterProjectBuilder.convertToFlutterProject (package:parabeac_core/generation/flutter_project_builder/flutter_project_builder.dart:127:35)

#10     Controller.convertFile (package:parabeac_core/controllers/controller.dart:55:15)

#11     SketchController.convertFile (package:parabeac_core/controllers/sketch_controller.dart:37:17)
#12     main (package:parabeac_core/main.dart:166:24)
#13     _RootZone.runUnary (dart:async/zone.dart:1450:54)
#14     _FutureListener.handleValue (dart:async/future_impl.dart:143:18)
#15     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:696:45)
#16     Future._propagateToListeners (dart:async/future_impl.dart:725:32)
#17     Future._complete (dart:async/future_impl.dart:519:7)
#18     _StreamIterator._onDone (dart:async/stream_impl.dart:1087:20)
#19     _RootZone.runGuarded (dart:async/zone.dart:1372:10)
#20     _BufferingStreamSubscription._sendDone.sendDone (dart:async/stream_impl.dart:410:13)
#21     _BufferingStreamSubscription._sendDone (dart:async/stream_impl.dart:420:15)
#22     _BufferingStreamSubscription._close (dart:async/stream_impl.dart:305:7)
#23     _SinkTransformerStreamSubscription._close (dart:async/stream_transformers.dart:97:11)
#24     _EventSinkWrapper.close (dart:async/stream_transformers.dart:23:11)
#25     _StringAdapterSink.close (dart:convert/string_conversion.dart:251:11)
#26     _Utf8ConversionSink.close (dart:convert/string_conversion.dart:305:20)
#27     _ConverterStreamEventSink.close (dart:convert/chunked_conversion.dart:82:18)
#28     _SinkTransformerStreamSubscription._handleDone (dart:async/stream_transformers.dart:142:24)
#29     _RootZone.runGuarded (dart:async/zone.dart:1372:10)
#30     _BufferingStreamSubscription._sendDone.sendDone (dart:async/stream_impl.dart:410:13)
#31     _BufferingStreamSubscription._sendDone (dart:async/stream_impl.dart:420:15)
#32     _BufferingStreamSubscription._close (dart:async/stream_impl.dart:305:7)
#33     _SyncStreamControllerDispatch._sendDone (dart:async/stream_controller.dart:816:19)
#34     _StreamController._closeUnchecked (dart:async/stream_controller.dart:671:7)
#35     _StreamController.close (dart:async/stream_controller.dart:664:5)
#36     _Socket._onData (dart:io-patch/socket_patch.dart:2051:21)
#37     _RootZone.runUnaryGuarded (dart:async/zone.dart:1384:10)
#38     _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:357:11)
#39     _BufferingStreamSubscription._add (dart:async/stream_impl.dart:285:7)
#40     _SyncStreamControllerDispatch._sendData (dart:async/stream_controller.dart:808:19)
#41     _StreamController._add (dart:async/stream_controller.dart:682:7)
#42     _StreamController.add (dart:async/stream_controller.dart:624:5)
#43     new _RawSocket. (dart:io-patch/socket_patch.dart:1587:35)
#44     _NativeSocket.issueReadEvent.issue (dart:io-patch/socket_patch.dart:1069:18)
#45     _microtaskLoop (dart:async/schedule_microtask.dart:41:21)
#46     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50:5)
#47     _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:118:13)
#48     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:169:5)
AB#755

According to your logs, parabeac-core could not find "flutter create". Have you ensured you have flutter installed and in your PATH?

I tried and it is breaking inside the code and doesn't have anything to do with flutter.
the 'source.managerData' for a particular node when processing that file is null
Ken

breaks on dev and stab le

For context, this issue may have to do with running parabeac-core on Big Sur, since it works well for Catalina.


runAsNonRoot

In general it's best practice to run processes as a non root user.
Because of that I've enabled checking for the securityContext settings to enforce it
I see in the clickhouse docker container it already does run as the user clickhouse but it's not enforced in the stateful set that gets created.
Would it have any adverse effects to add this to improve security?

We do not have it by default, but you can enable it in your ClickHouseInstallation:
    podTemplates:
      - name: pod-template-with-volumes-do-not-chown
        spec:
          securityContext:
            runAsUser: 101
            runAsGroup: 101
            fsGroup: 101
          containers:
            - name: clickhouse-pod
              image: yandex/clickhouse-server:20.8
              imagePullPolicy: IfNotPresent
              volumeMounts:
                - name: data-storage-vc-template-1
                  mountPath: /var/lib/clickhouse
              command:
                - clickhouse-server
                - --config-file=/etc/clickhouse-server/config.xml
              env:
                - name: CLICKHOUSE_DO_NOT_CHOWN
                  value: "1"

Finally got around testing it and works great. Thank you! since it's security best practice perhaps you might want to consider to enable this by default :-)


Add integration tests

This buildpack is missing any integration testing using pack and this buildpack to produce a container image.
We should add some simple integration tests that just assert that the php binary is added to an output image & that it correctly functions on the bionic stack.

Why bionic?
#43 adds integration tests, but the runner script integration.sh tests it on cflinuxfs3.
https://github.com/paketo-buildpacks/php-dist/blob/3ee02a05030e692be7d43e550ecdf6c01e1d5422/scripts/integration.sh#L50-L57


page body typo

[Enter feedback here]
typo this line:
<img height="?1"? width="?1"? sr\c=.?http://

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

ID: 37115a33-d073-cf0d-1506-2d2571cf1007
Version Independent ID: 07dd4b19-c5a5-1fba-cbad-3650922d357f
Content: Use mail flow rules to filter bulk email - Office 365
Content Source: microsoft-365/security/office-365-security/use-transport-rules-to-configure-bulk-email-filtering.md
Service: o365-seccomp
GitHub Login: @chrisda
Microsoft Alias: chrisda

@orbis-ming-chang, fixed as you described. In the future, you too can make fixes like this directly without opening a GitHub Issue first. For instructions, see https://github.com/MicrosoftDocs/microsoft-365-docs/blob/public/README.md.
Thanks for the feedback.
Closing


Docker!

☢☢☢ This is a work-in-progress ☢☢☢
This PR adds support for creating a docker container for Oscar's sandbox. Running docker/build.sh should create a container tagged oscar_latest that has the sandbox installed. You can then run Django's server within the container using:
$ docker run -it -p 8000:8000 /code/sites/sandbox/manage.py runserver <IP_ADDRESS>:8000

and browse it on your host machine. OSX users will have to forward port 8000 of the boot2docker VM too:
$ boot2docker ssh -L 8000:localhost:8000

Double port-forwarding! Yay
I'm relatively new to Docker so feedback is welcome.

As per the docker best practices it'd be nice if you separated each process into its own container then linked them all together using docker-compose.

I agree with @knicholes. It's strange seeing so much stuff crammed into one container.
My app is built with docker. This isn't it exactly, obviously, but an okay starting guide:
# docker-compose.yml
db:
  image: postgres
  volumes:
    - /var/lib/postgresql/pillowcentral:/var/lib/postgresql/data
memcached:
  image: memcached
elasticsearch:
  image: elasticsearch
nginx:
  build: deploy/nginx
  ports:
    - "8051:80"
    - "8052:443"
  volumes:
    - /var/www//example.com:/var/www/example.com
  links:
    - oscar
oscar:
  build: .
  command: gunicorn --config=gunicorn.py project.wsgi
  volumes:
    - /var/www/example.com:/var/www/example.com # for collectstatic and saving media
  links:
    - db
    - memcached
    - elasticsearch

# Dockerfile
FROM python:2.7
ENV PYTHONUNBUFFERED 1

RUN mkdir /code
WORKDIR /code

COPY requirements.txt /code/

RUN pip install -r requirements.txt

COPY code /code

RUN python manage.py collectstatic --noinput

@knicholes @john-parton Yes, I'm sure you're right. This PR was really me learning Docker back in 2014. Things have moved on since then but this PR hasn't.
I'm really sure it's worth having an official images for Oscar. It was a fun exercise but I'm not convinced they are that useful in general.

@john-parton @knicholes mb we could finalize this PR?

I actually have a dockerized oscar app running on top of Amazon's EC2 container service. It was a massive pain to set up, and there are some caveats, but it seems pretty solid so far.

I've just merged a simple implementation of docker for the sandbox via #1915 and closing this one. Please let me know if you don't agree with me :-)


modify Gan-introduction

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How has this been tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] local build the book
[ ] review the artifacts from local build
[ ] local build the slides if appliable
[ ] Unit test if appliable

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have applied tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Well, there are some problems to be solved:

every code block should follow black format, you can install black  plugins in VSCode.
I found some sentence follow upper style as a title/subtitle, it should be modified . Only the first character can be capital.
I found some subtitle are one size font  or too small , plz adjust them to the appropriate font size.
I found a lot of first-person expressions, plz change them.
In Acnowledge part, there is a spell wrong: oin->in, and a period (a full stop)  should be added after this sentence.
Thanks for your work.


jasmine_rails/application.js won't compile because of missing prototype.js

When trying to load http://localhost:3000/specs as per instructions in the README, I get this in my Rails server log:
2011-Dec-18 17:46:49 +0100 []: Error compiling asset jasmine_rails/application.js:

2011-Dec-18 17:46:49 +0100 []: Sprockets::FileOutsidePaths: {{local app path}}/public/javascripts/prototype.js isn't in paths: {{complete sprocket search path}}

I could fix this by adding the prototype-rails gem (which makes prototype.js (amongst other files) available in  the search path)) inside my test+development group of my gemfile, . But I guess I shouldn't need to.
I currently have zero Jasmine tests, so I can't really say if everything is working, but the base UI at http://localhost:3000/specs now loads up fine.
Interesting thing is that the log says jasmine_rails/application.js was compiled (sucessfully?) before.
All log messages generated by the request follow:
2011-Dec-18 17:46:48 +0100 []: Started GET "/specs" for <IP_ADDRESS> at 2011-12-18 17:46:48 +0100
2011-Dec-18 17:46:48 +0100 [action_controller]: Processing by JasmineRails::SpecRunnerController#index as HTML
2011-Dec-18 17:46:48 +0100 [action_view]: Rendered /Users/meryn/.rvm/gems/ruby-1.9.2-p290/gems/jasmine-rails-0.0.2/app/views/jasmine_rails/spec_runner/index.html.erb within layouts/jasmine_rails/spec_runner (0.3ms)

2011-Dec-18 17:46:48 +0100 []: Compiled jasmine_rails/application.css  (3ms)  (pid 3280)

2011-Dec-18 17:46:48 +0100 []: Compiled jasmine.css  (0ms)  (pid 3280)

2011-Dec-18 17:46:48 +0100 []: Compiled jasmine_rails/application.js  (12ms)  (pid 3280)

2011-Dec-18 17:46:48 +0100 []: Compiled jasmine.js  (0ms)  (pid 3280)

2011-Dec-18 17:46:48 +0100 []: Compiled jasmine-html.js  (0ms)  (pid 3280)

2011-Dec-18 17:46:48 +0100 []: Compiled json2.js  (2ms)  (pid 3280)
2011-Dec-18 17:46:48 +0100 [action_controller]: Completed 200 OK in 91ms (Views: 78.6ms | ActiveRecord: 0.0ms)

2011-Dec-18 17:46:49 +0100 []: Started GET "/assets/jasmine_rails/application.css?body=1" for <IP_ADDRESS> at 2011-12-18 17:46:49 +0100

2011-Dec-18 17:46:49 +0100 []: Served asset /jasmine_rails/application.css - 200 OK (0ms)

2011-Dec-18 17:46:49 +0100 []: Started GET "/assets/jasmine.css?body=1" for <IP_ADDRESS> at 2011-12-18 17:46:49 +0100

2011-Dec-18 17:46:49 +0100 []: Compiled jasmine.css  (0ms)  (pid 3280)

2011-Dec-18 17:46:49 +0100 []: Served asset /jasmine.css - 200 OK (46ms)

2011-Dec-18 17:46:49 +0100 []: Started GET "/assets/jasmine_rails/application.js" for <IP_ADDRESS> at 2011-12-18 17:46:49 +0100

2011-Dec-18 17:46:49 +0100 []: Compiled jasmine_rails/application.js  (12ms)  (pid 3280)

2011-Dec-18 17:46:49 +0100 []: Error compiling asset jasmine_rails/application.js:

2011-Dec-18 17:46:49 +0100 []: Sprockets::FileOutsidePaths: {{local app path}}/public/javascripts/prototype.js isn't in paths: {{complete sprocket search path}}

2011-Dec-18 17:46:49 +0100 []: Served asset /jasmine_rails/application.js - 500 Internal Server Error

Rails 3.1.1.

Interesting. I've tried against 3.1.1 and 3.1.3 both with the generated jasmine.yml and the modified jasmine.yml in the README, and have been unable to repro. Not sure where to go from here.

Me neither.
I think the main question is if in these (fresh?) Rails apps you have access to "prototype.js" (through Sprockets). I think by default, you should not. Maybe you get access to it when you have the rails-prototype gem installed globally (i.e. in your current ruby's gemset), not sure about that. I don't have it installed globally, so I had to include it explicitly in my gemfile.

I agree. Not sure if Sprockets gives access to prototype.js but let's call an explicit require the solution for now then, unless someone else can replicate?


[StyleCop] Fix all the issues in the AI.QnA.Tests project

Note: This PR is related to #1429 and must be merged after it.

Reorder methods
Add trailing comma
Fix spacing
Add a new file for a class

Replaced with #1488


EventHandlers and ActivityImpls should be full-blown workflow assets.

This means they should be handled as all other assets (appearing in .mdw/versions, etc).  However, since they only support a single version, they can be excluded from archiving.

migrated this issue to the mdw-designer repo.
closed the issue,removed the milestone, removed the designer label , removed the project  in this repo.


windows_comp.yml: Run openssl after it is built

This fixes windows_comp.yml CI regression from #24450

@nhorman any idea what's wrong with the script?

I'm trying to re-create it now.  Its really odd that its not producing any error output

ok with urgent

@t8m Err, which OTC member has approved this? (apologies if Neil is in the OTC, but I don't see his name at https://www.openssl.org/community/otc.html)

@t8m Err, which OTC member has approved this? (apologies if Neil is in the OTC, but I don't see his name at https://www.openssl.org/community/otc.html)

This was just a mistake.

Merged to the master branch as it is urgent CI fix. Thank you for the reviews.


ruby 2.6.3 build failing on Windows 10

running rbenv install 2.6.3 results in
Downloading ruby-2.6.3.tar.bz2...
-> https://cache.ruby-lang.org/pub/ruby/2.6/ruby-2.6.3.tar.bz2
Installing ruby-2.6.3...

BUILD FAILED (MSYS_NT-10.0 2.11.2(0.329/5/3) using ruby-build 20190423-4-gac92ec0)

Inspect or clean up the working tree at E:/temp-def/ruby-build.20190603180013.8084
Results logged to E:/temp-def/ruby-build.20190603180013.8084.log

Last 10 log lines:
                          -Wno-unused-value -Wsuggest-attribute=format \
                          -Wsuggest-attribute=noreturn -Wunused-variable
   * strip command:       strip
   * install doc:         yes
   * JIT support:         yes
   * man page type:       man

---
make: Nothing to be done for 'all'.
make: *** No rule to make target 'install'.  Stop.

complete log: https://gist.github.com/jpegzilla/307a3c93f9f0e936bad0fcb86b24db8b

Thank you for reporting! This looks like it might be a problem:
/usr/bin/sed: -e expression #1, char 18: unterminated `s' command

Can you share more about your Windows setup, i.e. which environment did you run ruby-build in? I'm guessing this is WSL?

it isn't exactly WSL. the environment is a bash shell running cygwin for access to unix-like utilities. the sed binary in question is the one included with the git bash for windows.
thanks for responding!

Downloading ruby-2.6.3.tar.bz2...
-> https://cache.ruby-lang.org/pub/ruby/2.6/ruby-2.6.3.tar.bz2
Installing ruby-2.6.3...

BUILD FAILED (Ubuntu 18.04 using ruby-build 20190615-7-g0e9094b)

Inspect or clean up the working tree at /tmp/ruby-build.20190714021939.2119
Results logged to /tmp/ruby-build.20190714021939.2119.log

Last 10 log lines:
        from /tmp/ruby-build.20190714021939.2119/ruby-2.6.3/lib/fileutils.rb:200:in `each'```

Related to #1337 ?
Is this still an issue?

sorry, I made this issue around a year ago and forgot about this. I no longer use rbenv. I was never able to fix it.

Thank you for the update!
Closing due to staleness


Fixed Automatic Episode Deletion

I've made it so that when the episode is seen and the video player is cleared, and episodes pending deletion is called, and also removed the coroutineScope due to it not working when it's there. (Note: I know little about coroutineScope, and I am unaware how important it is/if there is a replace in this scenario for it

In this case the courtoutine scope makes it run in a separate thread and as you see its set so even if the activity dies, it continues the deletion of episodes and doesn't get interrupted.
It basically helps make sure the process is independent of the player activity

Ah, didn't check branches. Thanks for the info about corountineScope!


JBPM-4913: Ability to read the underlying JNDI datasource name

This feature is required by the jBPM console in order to retrieve from the jBPM engine what is the data source JNDI name used by the datasets that feed the UI consoles.

+1, test failure not related and it going to be fixed with #382


Add athtech.txt

https://www.athtech.gr/
https://www.athtech.gr/en/courses/bachelors/computer-science/
-( it is under the same domain)

@apo-mak Pull request merged. Thank you!


How to pass the agent's SSH authentication socket?

I have a step using the docker-compose-buildkite-plugin, where I need to make operations on a private Git repository.
My idea was to pass the agent's SSH_AUTH_SOCK to the container (as the agent has access to the repo). As there is no mount-ssh-agent option in this plugin, I took a look at what that option does in the docker-buildkite-plugin, and tried to manually reproduce the behaviour on my compose stack by setting
...
 docker-compose#v3.0.0:
...
   environment:
     - SSH_AUTH_SOCK=/ssh-agent.sock
   volumes:
     - ${HOME}/.ssh/known_hosts:/root/.ssh/known_hosts:ro
     - ${SSH_AUTH_SOCK}:/ssh-agent.sock
...

My problem is, that while ${SSH_AUTH_SOCK} exists and is a socket on the agent, the mounted volume /ssh-agent.sock is a directory inside the container for some reason, and I cannot figure out why (note: this is on Linux, not on macOS).
Why is this happening?
Is there an easy way to pass the SSH authentication socket?

For the record, the best practice AFAIK is currently:
...
  docker-compose#v3.0.0:
...
    environment:
    - SSH_AUTH_SOCK=/ssh-agent
    volumes:
    - ${HOME}/.ssh/known_hosts:/root/.ssh/known_hosts:ro
    - /run/host-services/ssh-auth.sock:/ssh-agent
...

@jufemaiz : The solution os for OSX, I am running my steps on Linux.
I added the environment and volumes properties to the Composefile, and it is working for me.

What is the :ro at the end of the volume?
    - ${HOME}/.ssh/known_hosts:/root/.ssh/known_hosts:ro


Train on gpu predict on cpu?

Hello, everyone.
I'm new to Caffe2 and now working on my toy demo.
I wonder whether I can train my model with gpu and use(predict) it on cpu.
Any advice?

Same error here.


ga2.gaDisc.spread doesnt support floating point values

Currently , the internal random.random() function takes only integers as its input , which is not the case when spread becomes a floating point

Bug fixed


State history requiring a query and not a store

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[ ] Performance issue
[ ] Feature request
[x] Documentation issue or request
[ ] Support request
[ ] Other... Please describe:

Current behavior
We introduced Akita in our projects recently. The store and query abstractions provide a good way to encapsulate their respective functionalities. With that in mind, we wanted to also encapsulate the undo/redo. It felt natural to expose them as methods of the various Stores, as the documentation states that:

From the moment you call it, Akita's StateHistory tracks the history of the store

I was surprised, though, as the state history plugin requires a query. We ended up with methods like EntityQuery#undo(entityId), which feels out of place, as I think queries are supposed to only query.
Expected behavior
Could you elaborate (or document) why queries are needed for the state history, and not directly a store?

Because we need to listen to state changes:
https://github.com/datorama/akita/blob/master/libs/akita/src/lib/plugins/stateHistory/stateHistoryPlugin.ts#L91


[FEATURE REQUEST] Account specific proxy

Feature request: Have account specific proxies so each account is separated.

That's a great idea! It would make it possible to avoid account banning as much as possible!


seafile-shared: update source hash, seafile-client: update source hash

Motivation for this change
It looks like the tags have been modified
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 129111 run on x86_64-linux 1

2 packages built:

seafile-client
seafile-shared

Result of nixpkgs-review pr 129111 at 1a4c9851 run on aarch64-linux 1

2 packages built successfully:

seafile-client
seafile-shared

Backport would have required merging https://github.com/NixOS/nixpkgs/pull/129057 first, but it failed to build due to the outdated hashes. I will create a PR with both backports.


setup.py imports depedencies

Hi all!
Trying to start a new virtual env for python_dwd fails, because the setup.py has a line
from python_dwd import __version__
which triggers other imports from python_dwd/init.py and leads python to trying to import modules declared as dependencies in the setup.py. This way, pip has no chance to install the dependencies before they are imported.
Best,
Philip

sorry for double posting this, github gave me an http 500 the first time.

Fixed.


Watch not working: socket 1 did not return valid json

Running the command does not yield any useful information:
$ hyprwatch clients
socket 1 did not return valid json
socket 1 did not return valid json
socket 1 did not return valid json
socket 1 did not return valid json
socket 1 did not return valid json
socket 1 did not return valid json
socket 1 did not return valid json
socket 1 did not return valid json
socket 1 did not return valid json
socket 1 did not return valid json
socket 1 did not return valid json
socket 1 did not return valid json
socket 1 did not return valid json
socket 1 did not return valid json
socket 1 did not return valid json
socket 1 did not return valid json
socket 1 did not return valid json

Hyprland version:
Hyprland, built from branch  at commit 918d8340afd652b011b937d29d5eea0be08467f5  (flake.lock: update).
Date: Tue Jun 25 12:06:02 2024
Tag: v0.41.2, commits: 4886

flags: (if any)

should be fixed
did still split at the old characters, not these that were lately introduced (https://github.com/hyprwm/Hyprland/commit/546a486bab56af06fd95b23fb2ce231d00f671c9)

Thanks for the quick fix. I can confirm it's working now.


Patch 1

Ahoy, @Deepanshu0703!
Thank you for your interest in the project. We're trying to keep main branch separate from other changes (more details in CONTRIBUTING.md). Kindly rebase your commit on top of the changeme branch.
Cheers!


Function error not displayed

[x] Bug
[ ] Feature
[ ] Enhancement

Detailed Description

When an error occurs when running a function, the error is returned as part of the context but we don't display this to the user. It would be nice to display the error that occurred and set the function run status as "ERROR".
Currently when a function throws an exception,
def handle(ctx, payload):
    raise Exception("oh no!")

A traceback is returned in the stderr, but the function still reports as "READY" and reason is null
$ dispatch exec fail --wait
{
    "blocking": true,
    "executedTime":<PHONE_NUMBER>,
    "faasId": "00bbecbe-9815-44ce-8445-56715a107c6f",
    "finishedTime":<PHONE_NUMBER>,
    "functionId": "65f708fd-61b9-4949-af1d-fba822d3abea",
    "functionName": "fail",
    "input": {},
    "logs": {
        "stderr": [
            "Traceback (most recent call last):",
            "  File \"main.py\", line 43, in exec_function",
            "    r = handler.handle(msg['context'], msg['payload'])",
            "  File \"/function-server/function/handler.py\", line 2, in handle",
            "    raise Exception(\"oh no!\")",
            "Exception: oh no!"
        ],
        "stdout": null
    },
    "name": "19f2a8ae-6010-4ad5-98f5-5cd611f09101",
    "reason": null,
    "secrets": [],
    "services": null,
    "status": "READY",
    "tags": []
}

Context

Possible Implementation

Complexity

[x] Low - Simple enhancement or bug fix, no architectural changes or refactoring
[ ] Medium - Change requires some thought, but is relatively isolated
[ ] High - Significant architectural change or large refactor

Impact

[x] Low - Annoyance, but does not impact business or functionality
[ ] Medium - Issue can be worked around, but is causing pain
[ ] High - Blocker

Your Environment

Dispatch CLI version (or git commit):
Dispatch Chart version (or image[s] tag):
Operating System and version:
Kubernetes version (and distribution):
Etc (any other useful environmental information):

Related to #356

@rjew Is this done?

Not done yet. Working on it as part of the bigger error handling change.


Unregistered dependency on prop-types

It looks like this package requires prop-types, but doesn't define it in the package.json file. This causes errors if you try to install it into a project that doesn't already have prop-types.
You can see the import here: https://github.com/MetinArslanturk/react-animation-on-scroll/blob/main/dist/esm/components/AnimationOnScroll.js#L1.

https://github.com/MetinArslanturk/react-animation-on-scroll/pull/6 opened PR to address this issue.

Hey @AndrewCEmil thanks for your catch and pull request. Good one. It seems I missed it while copying some babel configs from another project.
It comes from: https://github.com/MetinArslanturk/react-animation-on-scroll/blob/50aac5c0fa17c9d902f30b05e058e8662a7c6b4d/.babelrc#L6
But I think we should get rid of from prop-types. This project is completely written in TypeScript and prop-types main purpose is a "type checking if you don't have typescript in development mode" (source). So, while we have TypeScript here, we don't need prop-types. I will remove it completely.
I will publish .d.ts files with the project files. So, it will be type-safe for typescript projects.

By the way, a fun fact, I think React already has prop-types dependency internally. So, you always having it even you don't have it in your package.json
But React contributors says don't trust this and include it in your package.json (source)

Now I released a new version (5.0.2) and removed prop-types completely. Again, thanks for catch!


Get rid of unnecessary comments

Idea
Comments are supposed to improve readability and make it easier to reason about the code. Currently there are a lot of comments that are redundant. There is no sense in commenting a function if its name clearly describes its purpose. There is no sense in commenting blocks of code in a large method, it is much better to extract them into their own functions.
Todo

If you see a comment that doesn't add important information, please remove it.
If you want to comment something, please consider another way of communicating what you're trying to achieve.

I agree that many comments in the repo add little to no value, can I take this issue?

I agree that many comments in the repo add little to no value, can I take this issue?

I agree that comments are supposed to improve readability, but we want the onboarding process for new contributors to be as quick as possible, and that is done through somewhat redundant comments. So, if you can find a way to make a redundant comment less redundant, do that, but don't remove comments that are already there.
https://mitcommlab.mit.edu/broad/commkit/coding-and-comment-style/
We are using the MIT guide for commenting, and on top of that, we have been using Google's idea that comments should break up code flow. What this means is in the example below, a comment should be used to break up two separate blocks of code, as it increases readability.
Bad:
// Create the Badge and Text
var resultTitle = "\033[1m";
if (result == true) resultTitle += "\033[1;37;42m PASSED ";
if (result == false) resultTitle += "\033[1;37;41m FAILED ";
resultTitle += "\033[22m";
resultTitle += "\033[0m";
return `${resultTitle} ${text}`;

Good:
// Set Title to Be Bold
var resultTitle = "\033[1m";
// Create the Badge Text
if (result == true) resultTitle += "\033[1;37;42m PASSED ";
if (result == false) resultTitle += "\033[1;37;41m FAILED ";
// Unbold Title
resultTitle += "\033[22m";
resultTitle += "\033[0m";
// Log to Console
return ${resultTitle} ${text};

I agree that comments are supposed to improve readability, but we want the onboarding process for new contributors to be as quick as possible, and that is done through somewhat redundant comments. So, if you can find a way to make a redundant comment less redundant, do that, but don't remove comments that are already there.
https://mitcommlab.mit.edu/broad/commkit/coding-and-comment-style/
We are using the MIT guide for commenting, and on top of that, we have been using Google's idea that comments should break up code flow. What this means is in the example below, a comment should be used to break up two separate blocks of code, as it increases readability.
Bad:
// Create the Badge and Text
var resultTitle = "\033[1m";
if (result == true) resultTitle += "\033[1;37;42m PASSED ";
if (result == false) resultTitle += "\033[1;37;41m FAILED ";
resultTitle += "\033[22m";
resultTitle += "\033[0m";
return `${resultTitle} ${text}`;

Good:
// Set Title to Be Bold
var resultTitle = "\033[1m";
// Create the Badge Text
if (result == true) resultTitle += "\033[1;37;42m PASSED ";
if (result == false) resultTitle += "\033[1;37;41m FAILED ";
// Unbold Title
resultTitle += "\033[22m";
resultTitle += "\033[0m";
// Log to Console
return ${resultTitle} ${text};

@William-McGonagle The point is to separate implementation details from higher level logic so you can get the general idea of what the method is doing without looking at the concrete implementation. Conversely, if you need to see how the function is implemented you look at it. I often extract the commented blocks into functions that are named after the comments. Two birds with one stone.

@William-McGonagle The point is to separate implementation details from higher level logic so you can get the general idea of what the method is doing without looking at the concrete implementation. Conversely, if you need to see how the function is implemented you look at it. I often extract the commented blocks into functions that are named after the comments. Two birds with one stone.

@NikitaLejnev I do have to say- I really like the new utility functions that you have added as they reduce the need for comments, but in the places like Sequelize references or high-level express stuff, we should keep them in because it makes the code easier for the new commiters.

@NikitaLejnev I do have to say- I really like the new utility functions that you have added as they reduce the need for comments, but in the places like Sequelize references or high-level express stuff, we should keep them in because it makes the code easier for the new commiters.

@William-McGonagle Fair enough. Thank you for being open to discussing ideas and suggestions. I like your idea and love the way you handle the project.

@William-McGonagle Fair enough. Thank you for being open to discussing ideas and suggestions. I like your idea and love the way you handle the project.

@William-McGonagle, possibly I'm not understanding correctly, but my assumption for this issue was for removing comments that don't add value.
A example I found in the code:
`function TokenizeString(input) {
// Make it Lowercase
const lowercase = input.toLowerCase();
// Switch Around Similar Looking Symbols
let partsRemoved = lowercase;
partsRemoved = partsRemoved.replace(/$/g, "s");
partsRemoved = partsRemoved.replace(/@/g, "a");
// Return the Final Result
return partsRemoved;
}`
Here the top and bottom comments are not adding value because the following line describes exactly the same thing.
But the middle comment adds value as it is not immediately obvious what that part of the code does.
Just my 2 cents, but I would like to keep working on this project, as I like the idea you've got here.

@William-McGonagle, possibly I'm not understanding correctly, but my assumption for this issue was for removing comments that don't add value.
A example I found in the code:
`function TokenizeString(input) {
// Make it Lowercase
const lowercase = input.toLowerCase();
// Switch Around Similar Looking Symbols
let partsRemoved = lowercase;
partsRemoved = partsRemoved.replace(/$/g, "s");
partsRemoved = partsRemoved.replace(/@/g, "a");
// Return the Final Result
return partsRemoved;
}`
Here the top and bottom comments are not adding value because the following line describes exactly the same thing.
But the middle comment adds value as it is not immediately obvious what that part of the code does.
Just my 2 cents, but I would like to keep working on this project, as I like the idea you've got here.

@Nokkvi I would agree with your point that the first and last comment don't add value. But you are right that the middle comments should be kept because they somewhat help to explain the flow.

@Nokkvi I would agree with your point that the first and last comment don't add value. But you are right that the middle comments should be kept because they somewhat help to explain the flow.

#93


Add offline execution via Microsoft.Fx.Portability.Offline

These changes add the following (optional) components to ApiPort -

Microsoft.FX.Portability.Offline (for offline analysis)
A few seed breaking change files to allow for offline breaking change analysis (the expectation is that more breaking change files will be added as UE and subject matter experts review them)
The HTML and Json report writers for offline use (including a couple minor bug fixes).

With these changes, users can inject offline components (via Unity.config) and run ApiPort entirely offline.

It's a pretty big review, so if there's anything you want a careful review of, let me know.  I looked through and didn't see anything that stood out to me, so looks good.

Yea, the review is big since I'm porting some components that were previously built internally. Probably the best way to look at it is to look at individual commits and skip the first and third ones (since those are just copying already-existent assets wholesale).

That helps a lot!  Why are we downloading the breaking changes since they are in the same repo?  Can't we just use the ones already in the repo?

Good point :) There was a time when they weren't in this repo, but I decided to just move them. I'll update things to use those instead of re-downloading from Github.

I would prefer embedding the md files instead of having them next to it so that the offline library can be self contained

Yea, I can see the value of the library being self-contained. On the other hand, though, I think it's important the users be able to override which breaking changes they're looking for (either to remove some they don't care about or add others).
Perhaps I can embed them and drop them next to the library (so that it's obvious that's an option) and update the library to fall back to embedded ones if none are found externally?

That's what I've done in the past - that way the library itself is functional, but can be augmented with new info easily.  If two with the same ID are available, we should prefer the one in the directory (ie it supersedes the embedded version)

Ok, I've updated the PR to work that way.

Looks like it will only take one or the other.  That's fine, but we may want to look at being selective (like I commented above) at some point.  Looks good to me!

I made it just use one or the other intentionally, that way users can easily remove breaking changes from the set of what's scanned for as well as add. As you say, we can look at adding selectively if that becomes an important scenario (wouldn't be too big of a change).


Bug: Skills field doesn't accept skills with space

Unable to add person because "time management" has a space

This is a duplicate issue

I will close this since this is fixed


Update lunatic 2.6.3

Update lunatic dependency to 2.6.3
Handle the new error structure (array instead of object)
Handle the error for the ComboLabelSelection component
Add role="combobox" on the suggester input
bump version to 1.0.9

LGTM thank you


Create a "veefind" tool to take the place of "veels", based on find rather than ls

As Ben pointed out, we should be using find rather than ls to iterate over files.

not gonna happen


Proposal: Use conventions to define the schema path

Hi,
Recently I make a proposal to create a test helper on the AMS, that follows the same idea, use a schema file to validate a API response. But in my case I am using conventions to load the schema file. Example
describe "GET /posts" do
  it "returns Posts" do
    get posts_path, format: :json

    expect(response.status).to eq 200
    expect(response).to match_response_schema
  end
end

I remove the param of the schema path, and assumes that the schema file lives in spec/support/api/schemas/posts/index.json since it is the posts#index tests and make the param optional. Makes sense to you guys?
A little doubt, Makes sense this be a matcher on the shoulda-matchers? This way we will need only a dependency and we can use this with the Minitest too. I think we can handle this as a Action Controller matcher. @mcmire makes sense?
Thanks guys

@maurogeorge Thanks for opening this ticket.
This approach is very interesting. I'm curious if this convention could be inferred outside a Rails app (Sinatra, Grape, or other Rack apps).
Would you be willing to flesh out the implementation you're considering?

@seanpdoyle Yeah I really don't know if this convention could be inferred outside of Rails. But in any case we can start doing this behavior only on Rails apps and in other Rack apps the user need to pass the params explicitly. What you think?

@maurogeorge That's true. I think it depends on how robust the implementation is, and the sorts of patterns consumers use for declaring their schema.
Would you be willing to work a PR with an initial implementation  or some failing test cases?

This approach sounds interesting, but to me it sounds a little magical. I tend to avoid magic as much as I can in my tests, and I really like the current approach that we're taking with this matcher because it's easy to see which schema file is going to be referenced. If the test fails in some manner and the test doesn't have some kind of hint as which file contains the error, then it's going to be difficult to debug the test, especially for people who are not familiar with this matcher and using it for the first time. (I like to make tests as easy to debug as possible too.)
As far as merging this in with shoulda-matchers, we intentionally decided to keep this separate as we didn't really know if it would fit in and we figured we'd have to play around with it first internally. We may decide to merge it in the future, but for now I think it's best if we keep it separate.

@seanpdoyle Yeah I can work on a PR, maybe in the end of the next week, but after the @mcmire comments do you belive that's make sense this PR or prefer keep the things the way it is?
@mcmire thanks for your feedback. I understand your point, make the things explicit has values too, it is about trade-offs, if the file is a param we can have a folder with all schemas or a confusing organization, following a convention we try to avoid this.
About the shoulda-matchers thanks for clarifying :smile:


Menu item for PNG zoom factor missing

Hello everbody
When using the desktop or online version of diagramm.net for editable PNGs, I have a menu item "Properties..." available in the File menu which allows to define the zoom factor for PNG rendering. When using a typical default font size like 12 and line thinkness like 1, an image rendering with 100% produces really tiny PNGs. So usually I set up a rendering zoom factor of 400% or so to produce reasonably sizes images.
It seems that this menu item is missing in the plugin. I'm working with plugin version 0.1.11 and intelliJ Ultimate 2020.3

I have the same issue. Would be great if the PNG output resolution can be increased somehow...

now I have the same issue, but I don't find any information on how to fix this. :-|


ftp-deploy does not work behind a proxy

Everything works, bower, npm, git, but this task fails behind a proxy.
How can this be fixed?

FTP behind a proxy can be a pain. This task uses jsftp as its transport level. I think jsftp is a better place to see if you can find something that would resolve your problem.


Update of pip

Maybe it's worth to avoid this repeated hint by updating pip in next AIO release:
[notice] A new release of pip is available: 23.3.2 -> 24.0
[notice] To update, run: D:\RobotFramework\python39\python.exe -m pip install --upgrade pip

Hi Holger,
Based on my check, we've already updated the latest pip package on AIO. I think that the issue with your output log occurred  because it redirected to a different location for the pip package.
Could you help me to check the output log for the command below on your machine?
D:\RobotFramework\python39\python.exe -m pip --version

Thank you and best regards,
Thong Hua

Hi  @HolQue ,
can you please check.
Thank you,
Thomas

Hi Thong,
on my machine it's:
pip 23.3.2 from D:\RobotFramework\python39\lib\site-packages\pip (python 3.9)

Hi Holger,
the problem is that you have to run our setup scripts (cloneall?) from beginning.
With rebasing of existing repos there will be no update of the python. This will be only one time created at setup time of the whole development environment.
Our build and SelfTests install always all new at new virtual machines, therefore there the latest version will be used.
Thank you,
Thomas

Hi Thomas,
this I do not understand. I use the RobotFramework AIO installer and expect to get the current state of all components installed.

Hi @HolQue ,
O.K. got the point!
@huavanthong : when I install RobotFramework AIO new with our windows and/or deb installer, then I expect also the latest pip version. Can you please check this.
Thank you,
Thomas

Hi Thomas, Holger
The AIO package've been already installed the latest pip package.
It can be found at
https://github.com/test-fullautomation/RobotFramework_AIO/blob/a859e7f7039c3b0b82baf1a7b092c553a966a2aa/install/install.sh#L302
Therefore, we will get the latest pip package in the next AIO release.
However, I noticed that the Linux platform has not been updated to the latest version yet, so I would like to create a pull request https://github.com/test-fullautomation/RobotFramework_AIO/pull/244 for this.
Please take it for your info.
Thank you and best regards,
Thong Hua

very good!
I already merged #244


Updates on the evaluation function

@gityves Made a new one now without conflicts, since #36 has been merged


Automatically open specific feed upon startup

yaymukund on IRC just asked if it's possible to automatically open a specific feed when Newsboat starts up. They aren't the first one to request that, see https://github.com/akrennmair/newsbeuter/issues/402 for the previous discussion.

This will be possible in the next releaseof Newsboat (or by building from the current master branch).
You can use the run-on-startup option in the config file to select a feed based on (part of) its title and then open it, e.g.:
run-on-startup      goto-title "newsboat" ; open


Error in orientdb independent transaction in multiple connection of the same pool

OrientDB Version: 2.2.29
Java Version: <IP_ADDRESS>
OS: GNU/Linux
Expected behavior
A commit in independent transactions of the same pool save the data related to each one.
Actual behavior
A single commit save all data from all Tx
Steps to reproduce
`
OrientGraphFactory fact = new OrientGraphFactory("remote:localhost/Test", "root", "toor").setupPool(1, 10);
OrientGraph t1 = fact.getTx();
OrientGraph t2 = fact.getTx();

t1.addVertex("class:Test", "text","transac 1");
t2.addVertex("class:Test", "text","transac 2");

t1.commit();

t2.rollback();

t1.shutdown();
t2.shutdown();

`
At the end, both record are added to the database, but I want only t1. Why? I think the getTx() return an independent connection to the DB but it seems they are all linked.

hi @mdre,
OrientDB associate the graph instance to the thread, and it use that association, so you should not use multiple graph instances on the same thread, if you really need to use two instance in the same thread you can use graph.makeActive() API before do an operation on the specific graph instance.
Regards
Emanuele

Thank's @tglman.
I will see if I can fix this in some way. What I need is to have every window bounded to it's own tx() so I can commit/rollback any change in a simple way.
Sincerely
Marcelo

Thank's @tglman.
I will see if I can fix this in some way. What I need is to have every window bounded to it's own tx() so I can commit/rollback any change in a simple way.
Sincerely
Marcelo

Thank's @tglman.
It's not clear why exist the getTx(). The way it is implemented seems not useful. If I have a pool instance is in one thread, but the Tx depend on the thread so the pools is useless.
I will see if I can fix this in some way. What I need is to have every window bounded to it's own tx() so I can commit/rollback any change in a simple way.
Sincerely
Marcelo

Hi @mdre,
The pool is useful for multithread, if you have just one thread doing the operation to the database it can use everytime the same session.
Regards


couch base compatibility

Is this considered compatible with couch base, since it uses a very similar protocol to couchdb? How hard would it be to test this?

CoverCouch uses nano to communicate with CouchDB. Is nano compatible with couchbase? Don‘t think so.


Feed pre-trained embeddings to NVTabular

What is your question?
I have a dataset that includes a column feature of pre-trained embeddings. I couldn't find any documentations or examples on how this column should be passed to NVTabular. Is it treated as a continuous feature?

@MelissaKR thanks for the question. Can you tell us more about your use case?
Basically, if you want to do some feature pre-processing on the column of pre-trained embeddings, yes, you can feed them as continuous features to NVTabular.
Let us know if you have further questions.

@rnyak Thank you for your response. I basically have another model that outputs embeddings for a given set of features, and I want to replace those features in the original model with the embeddings I have obtained.
Should I simply pass these new feature columns as conts in TorchAsyncItr? It'll be great if I could see an example code of how pre-trained embeddings are passed to NVTabular's TorchAsyncItr.


Shift+Tab on single line to remove starting tab

Resolves #149

Seems like the Netlify app config update is required, an old version of the build image is no longer supported.

@bombillazo thx!


Update to v0.5.0

orjson -> cargo-bundle-licenses
Package Info
Home: https://github.com/sstadick/cargo-bundle-licenses
Development: https://github.com/sstadick/cargo-bundle-licenses
Documentation: https://github.com/sstadick/cargo-bundle-licenses

Linter check failed, with the following output:
The following problems have been found:
ERROR: /tmp/abs_ea9p02hrfc/clone/recipe/meta.yaml:0: missing_tests: The recipe is missing tests.
WARNING: /tmp/abs_ea9p02hrfc/clone/recipe/meta.yaml:60: missing_doc_source_url: The recipe is missing a doc_source_url
WARNING: /tmp/abs_ea9p02hrfc/clone/recipe/meta.yaml:60: missing_license_url: The recipe is missing a license_url
Errors were found

Linter check failed, with the following output:
The following problems have been found:
ERROR: /tmp/abs_9b2311pk2i/clone/recipe/meta.yaml:0: missing_tests: The recipe is missing tests.
WARNING: /tmp/abs_9b2311pk2i/clone/recipe/meta.yaml:61: missing_license_url: The recipe is missing a license_url
Errors were found

The following changes might be recomended:
license_url: https://github.com/sstadick/cargo-bundle-licenses/blob/main/LICENSE-APACHE license_url: https://github.com/sstadick/cargo-bundle-licenses/blob/main/LICENSE-MIT
doc_source_url: https://github.com/sstadick/cargo-bundle-licenses/blob/main/README.md
might need to change the doc_url to the following:
doc_url: https://github.com/sstadick/cargo-bundle-licenses/blob/main/README.md

Can you use two license URLs? The linter crashes when you do and since it's not a hard requirement, I would skip this for now.
I think the doc_url is good the way it is. It points to the same file anyway.

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_35lje00960/clone/recipe/meta.yaml:0: missing_tests: The recipe is missing tests.
WARNING: /tmp/abs_35lje00960/clone/recipe/meta.yaml:61: missing_license_url: The recipe is missing a license_url
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_86emvgvoqz/clone/recipe/meta.yaml:0: missing_tests: The recipe is missing tests.
WARNING: /tmp/abs_86emvgvoqz/clone/recipe/meta.yaml:61: missing_license_url: The recipe is missing a license_url
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_8dewcbu7dr/clone/recipe/meta.yaml:0: missing_tests: The recipe is missing tests.
WARNING: /tmp/abs_8dewcbu7dr/clone/recipe/meta.yaml:61: missing_license_url: The recipe is missing a license_url
Errors were found


Find a more descriptive name for the spatial light modulator size

          Is there a more descriptive name that could specify what type of size this is? Perhaps `pixel_size`? Also, what pixels in particular are being referenced for this? (similar field usually refers to the pixels on a display being shown to a subject, for instance)

Originally posted by @CodyCBakerPhD in https://github.com/catalystneuro/ndx-patterned-ogen/pull/2#discussion_r1456324748

changed in spatial_resolution: https://github.com/catalystneuro/ndx-patterned-ogen/blob/8a8c8af6c589bf675565598007c561fcb599eddb/src/pynwb/ndx_patterned_ogen/patterned_ogen.py#L28-L34
it seems to be the correct term indicating the dimension in pixels: SLM:Trends and Applications

Change with spatial_resolution (Spatial Light Modulators: Trends and Applications):
https://github.com/catalystneuro/ndx-patterned-ogen/blob/8a8c8af6c589bf675565598007c561fcb599eddb/src/pynwb/ndx_patterned_ogen/patterned_ogen.py#L28-L34


Forgotten Updates of English Parts to Portuguese

Some of the bolded words hadn’t been translated from “User Story” to “História do Usuário”, capitalized the word Usuário for consistency.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thanks for your contribution and happy codding :smiley_cat:


Route Provider & Config files.

New route style like laravel 5.3+ (folder with api & web routes separated).

Fixed routes:

removed from the master service provider
added a route service provider

Allow multiple config files in a module. All will be append to the master config array.

Hello,
Thank you for the contribution and the time you took to make this pull request.
However, I can't accept this as it's unclear with it provides, while also being a breaking change.


⚠ Bofrost.at has degraded performance

In 863f800, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 845 ms

Resolved: Bofrost.at performance has improved in dc22ece after 6 minutes.


Spacing between header and cell in Table view. The TableView does not display from start of activity

Hi,
I'm using custom TableView. I'm facing the issue that there is a space between column header and cell view. and the TableView does not display from start of activity. I set TableView's width to "match_parent" and activity's width to "match_parent" too.
Please help me to fix this. Thanks!

Hi @namtran4194 Which version do you use?

i use version 0.8.8 and the layouts and holders are based on your sample app

Could you please share some code?

okie, i will share it tomorrow

Hi, here some code, please take a look.
https://github.com/namtran4194/smd

I guess you tried to hide row header. But unfortunately, you can not hide the row headers. Because it helps to the TableView in some calculations.
    @Override
    public AbstractViewHolder onCreateRowHeaderViewHolder(ViewGroup parent, int viewType) {
        return null;
    }

    @Override
    public void onBindRowHeaderViewHolder(AbstractViewHolder holder, Object rowHeaderItemModel, int rowPosition) {

    }

    @Override
    public View onCreateCornerView() {
        return null;
    }

Yeah, i want to hide row header because i don't need it. So how can i fix this?

After i add row header, corner view and fixed width - height for row header. All issues are gone.
Hope you work around to make TableView actually fit 'match_parent'

After i add row header, corner view and fixed width - height for row header. All issues are gone.
Hope you work around to make TableView actually fit 'match_parent'

Could you please share that clculation code?

After i add row header, corner view and fixed width - height for row header. All issues are gone.
Hope you work around to make TableView actually fit 'match_parent'

Could you please share that clculation code?

Adding below lines in dimens.xml solved the issue
<dimen name="default_row_header_width">55dp</dimen>     <dimen name="default_column_header_height">40dp</dimen>

Adding below lines in dimens.xml solved the issue
<dimen name="default_row_header_width">55dp</dimen>     <dimen name="default_column_header_height">40dp</dimen>


Package install hangs

I am trying to create a new virtual env and install a specific package (paramiko==<IP_ADDRESS>), but it seems that every time I do that, the installation hangs in a specific point.
My setup:
import shutil

from virtualenvapi.manage import VirtualEnvironment

if __name__ == '__main__':
    venv = VirtualEnvironment('./bla-env')
    venv.install(('paramiko', '<IP_ADDRESS>'), options=['--no-cache'])
    print venv.installed_packages
    shutil.rmtree('./bla-env')

Using PyCharm, I have been able to track down where it hangs:
Traceback (most recent call last):
  File "/home/alacea/pycharm-2018.3.5/helpers/pydev/pydevd.py", line 1741, in <module>
    main()
  File "/home/alacea/pycharm-2018.3.5/helpers/pydev/pydevd.py", line 1735, in main
    globals = debugger.run(setup['file'], None, None, is_module)
  File "/home/alacea/pycharm-2018.3.5/helpers/pydev/pydevd.py", line 1135, in run
    pydev_imports.execfile(file, globals, locals)  # execute the script
  File "/home/alacea/test-venv/src/vapi-test.py", line 7, in <module>
    venv.install(('paramiko', '<IP_ADDRESS>'), options=['--no-cache'])
  File "/home/alacea/test-venv/env/lib/python2.7/site-packages/virtualenvapi/manage.py", line 223, in install
    self._execute_pip(['install'] + package_args + options)
  File "/home/alacea/test-venv/env/lib/python2.7/site-packages/virtualenvapi/manage.py", line 127, in _execute_pip
    return self._execute(exec_args, log=log)
  File "/home/alacea/test-venv/env/lib/python2.7/site-packages/virtualenvapi/manage.py", line 137, in _execute
    output, error = proc.communicate()
  File "/opt/test/base/lib/python2.7/subprocess.py", line 483, in communicate
    return self._communicate(input)
  File "/opt/test/base/lib/python2.7/subprocess.py", line 1124, in _communicate
    stdout, stderr = self._communicate_with_poll(input)
  File "/opt/test/base/lib/python2.7/subprocess.py", line 1178, in _communicate_with_poll
    ready = poller.poll()
KeyboardInterrupt

Do you have any idea what might be causing this?
Installing the same package from the CLI (--no-cache as well) works just fine. I didn't come across other packages causing this so far.

Apparently PyCharm is actually the culprit here. Running the same script from the CLI works fine. I'll investigate further & i'll come back if there's anything relevant to this package.

Fixed, nothing related to the virtualenv-api package. I'll just leave this here if anyone comes across the same issue. The fix can be found at: https://github.com/fabric/fabric/issues/958#issuecomment-60308187

Nice catch! Glad you figured it out.


feat: add kubernetes credential hook for CLI

This commit introduces GetCredential function, which extracts authorization credential from kubernetes client. Using this way, we can:

create kubeclient with user credential in serverside (which allows us to leverage kubernetes native RBAC)
get credential from various providers (e.g. GKE)

In-line comments aside, I have a broader concern here in that I cannot seem to make this work.
Example code:
func main() {
	cfg, err := clientcmd.BuildConfigFromFlags("", "/Users/krancour/.kube/config")
	if err != nil {
		log.Fatal(err)
	}

	token, err := kubeclient.GetCredential(context.Background(), cfg)
	if err != nil {
		log.Fatal(err)
	}

	fmt.Println(token)
}

Debugging you underlying implementation, it seems no Authorization header is set on the outbound request.
I've tried this using both kind and k3d.

Researching more, it seems in the case of both kind and k3d, clients authenticate using a certificate rather than an auth header.
The key takeaway may be that there isn't a universal way to acquire credentials that will allow the API server to impersonate a user.
Maybe we need a multi-pronged approach to gleaning those credentials?

Adding to this that @jessesuen believes beyond local clusters like kind or k3d, cert-based authn is a less common scenario than token-based authn.
So I think this PR gets us probably halfway to where we need to be and accommodations for clients that use certs can be made in a follow-up.

Researching more, it seems in the case of both kind and k3d, clients authenticate using a certificate rather than a bearer token in an auth header.
The key takeaway may be that there isn't a universal way to acquire credentials that will allow the API server to impersonate a user.
Maybe we need a multi-pronged approach to gleaning those credentials?
Ref: https://kubernetes.io/docs/reference/access-authn-authz/authentication/

I think we may use this way for now like the kubernetes-dashboard, which is the kind of de-facto for proxing user credential 🤔

@devholic I just tried this little experiment:
package main

import (
	"fmt"
	"log"

	"k8s.io/client-go/tools/clientcmd"
)

func main() {
	cfg, err := clientcmd.BuildConfigFromFlags("", "/Users/krancour/.kube/config")
	if err != nil {
		log.Fatal(err)
	}

	if cfg.BearerToken != "" {
		fmt.Println("Found bearer token in configuration")
	} else if cfg.BearerTokenFile != "" {
		fmt.Printf("Found bearer token in file %s\n", cfg.BearerTokenFile)
	} else {
		if cfg.TLSClientConfig.CertData != nil {
			fmt.Println("Found cert in configuration")
		} else if cfg.TLSClientConfig.CertFile != "" {
			fmt.Printf("Found cert in file %s\n", cfg.TLSClientConfig.CertFile)
		}

		if cfg.TLSClientConfig.KeyData != nil {
			fmt.Println("Found key in configuration")
		} else if cfg.TLSClientConfig.KeyFile != "" {
			fmt.Printf("Found key in file %s\n", cfg.TLSClientConfig.KeyFile)
		}

		if cfg.TLSClientConfig.CAData != nil {
			fmt.Println("Found CA in configuration")
		} else if cfg.TLSClientConfig.CAFile != "" {
			fmt.Printf("Found CA in file %s\n", cfg.TLSClientConfig.CAFile)
		}
	}
}

This found everything (but still doesn't account for every authn method). I am wondering if we could use logic like this to find whatever form of credentials happen to be used. We can base64 encode, pack them into a struct that has separate fields for tokens, certs, etc., marshal that to JSON, base64 encode the JSON and pop all of that into an auth header.
Our API server can reverse the process -- base64 decode, unmarshal, see which field(s) aren't empty, decode their contents, and use those credentials to reconstruct rest client configuration OR TLSClientConfig.